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.

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

fix(gw): missing return if dir fails to finalize #8806

Merged
merged 1 commit into from
Apr 8, 2022

Conversation

makew0rld
Copy link
Contributor

Just fixing a small bug I noticed when working on a project for Agregore Mobile, something I discussed with @lidel.

@welcome

This comment was marked as resolved.

@lidel lidel changed the title Add missing return if dir fails to finalize fix(gw) missing return if dir fails to finalize Mar 21, 2022
@lidel lidel changed the title fix(gw) missing return if dir fails to finalize fix(gw): missing return if dir fails to finalize Mar 21, 2022
@lidel lidel self-assigned this Mar 21, 2022
Copy link
Member

@lidel lidel left a comment

Choose a reason for hiding this comment

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

LGTM, but not sure why the go-ipfs-http-client test fails consistently.

Let's wait with merge until CI can be made green.

@lidel lidel added the status/blocked Unable to be worked further until needs are met label Mar 21, 2022
@makew0rld
Copy link
Contributor Author

Thanks. Any idea if the failure due to my code, or if there's some CI issue? The failing tests seem unrelated to these changes.

@BigLep BigLep assigned makew0rld and unassigned lidel Mar 23, 2022
@BigLep BigLep added this to the Best Effort Track milestone Mar 23, 2022
@BigLep
Copy link
Contributor

BigLep commented Mar 25, 2022

You're right. This is a go-ipfs CI issue we're actively getting fixed. We'll merge after.

@lidel lidel removed the status/blocked Unable to be worked further until needs are met label Apr 6, 2022
@lidel
Copy link
Member

lidel commented Apr 6, 2022

Rebased on master, let's wait for CI.

@lidel lidel merged commit 52bf133 into ipfs:master Apr 8, 2022
xrazis pushed a commit to xrazis/kubo that referenced this pull request Apr 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
topic/gateway Topic gateway
Projects
No open projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants