Skip to content

chore(gw): extract logical functions to improve readability#8855

Closed
justindotpub wants to merge 1 commit intoipfs:masterfrom
fission-codes:justincjohnson/redirects-refactoring-1
Closed

chore(gw): extract logical functions to improve readability#8855
justindotpub wants to merge 1 commit intoipfs:masterfrom
fission-codes:justincjohnson/redirects-refactoring-1

Conversation

@justindotpub
Copy link
Copy Markdown
Contributor

@justindotpub justindotpub commented Apr 7, 2022

@lidel if you approve, this is an initial PR with some cleanup before implementing any redirects support.

@justindotpub justindotpub requested a review from lidel as a code owner April 7, 2022 14:09
@justindotpub justindotpub mentioned this pull request Apr 7, 2022
Closed
@justindotpub justindotpub force-pushed the justincjohnson/redirects-refactoring-1 branch from 957fc24 to ec10aa3 Compare April 12, 2022 12:46
chore: gateway_handler.go
01bd73c 
Extract functions from getOrHeadHandler to improve readability and prepare for later refactorings
@justindotpub justindotpub force-pushed the justincjohnson/redirects-refactoring-1 branch from ec10aa3 to 01bd73c Compare April 12, 2022 13:30
@lidel lidel changed the title chore(gateway): Extract functions from getOrHeadHandler to improve readability chore(gw): extract logical functions to improve readability Apr 13, 2022
@lidel lidel mentioned this pull request Apr 13, 2022
Closed
3 tasks
@lidel
Copy link
Copy Markdown
Member

lidel commented Apr 13, 2022 •
edited
Loading

@justincjohnson permissions at fission-suite org make it very difficult to review your PRs:

  • CircleCI does not run because of your org permissions
  • I am not able to push small cleanups

To work around this, I am closing this PR and re-opening as upstream branch in #8883 – let's continue there.

Not sure how to avoid these problems In the future 🤔
Would it be possible for you to open PRs from your personal account?
It should fix both CI and review papercuts. 🙏

@lidel lidel closed this Apr 13, 2022
@justindotpub
Copy link
Copy Markdown
Contributor Author

justindotpub commented Apr 13, 2022

No problem @lidel. Sorry for the bumps that have made this review process more painful.

@justindotpub
Copy link
Copy Markdown
Contributor Author

justindotpub commented Apr 14, 2022

@lidel Fission prefers that we work on branches in our org. By any chance are the CircleCI permission issues documented somewhere and something we could just fix on our end, or are there other problems caused by us working in our own org? Thanks.

@lidel
Copy link
Copy Markdown
Member

lidel commented Apr 14, 2022

@justincjohnson not aware of any docs, this seems to be uncharted teritory. You could see if https://app.circleci.com/ allows you to enable CircleCI on your go-ipfs fork repo. Perhaps that will solve the problem?

@justindotpub justindotpub mentioned this pull request Apr 20, 2022
Closed
@lidel lidel mentioned this pull request Apr 29, 2022
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@lidel lidel Awaiting requested review from lidel lidel is a code owner

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@justindotpub @lidel