Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

fix(core): return None if metadata unavailable #4613

Merged
merged 1 commit into from
May 15, 2024
Merged

fix(core): return None if metadata unavailable #4613

merged 1 commit into from
May 15, 2024

Conversation

NKID00
Copy link
Contributor

@NKID00 NKID00 commented May 14, 2024

Closes #4609.

@NKID00 NKID00 requested a review from Xuanwo as a code owner May 14, 2024 14:43
@github-actions github-actions bot requested a review from silver-ymz May 14, 2024 14:43
Xuanwo
Xuanwo approved these changes May 15, 2024
View reviewed changes
Copy link
Member

@Xuanwo Xuanwo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Perfect, thanks!

@Xuanwo Xuanwo merged commit 51f1f45 into apache:main May 15, 2024
214 checks passed
George-Miao pushed a commit to George-Miao/opendal that referenced this pull request Jun 5, 2024
@NKID00 @George-Miao
fix(core): return None if metadata unavailable (apache#4613)
a28804a
@NKID00 NKID00 deleted the fix-4609 branch July 8, 2024 08:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Xuanwo Xuanwo Xuanwo approved these changes

@silver-ymz silver-ymz Awaiting requested review from silver-ymz

Assignees
No one assigned
Labels
releases-note/fix
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

hide Metadata assert insides opendal's own tests features
2 participants
@NKID00 @Xuanwo