-
Notifications
You must be signed in to change notification settings - Fork 9
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
Rename and move to ipfs/ org #80
Comments
Absolutely. Would it make sense to live at ipshipyard/service-worker-gateway? |
Probably just a typo - I would have though it'd be https://github.com/ipfs/helia-service-worker-gateway ? |
Ah, you mean the org |
It is a good opportunity to agree on both :-) See #31 (comment) – suggestion is to remove With that in mind, my suggestion is to go with simple and self-explanatory If we want a made-up name, we could call it |
I think using Helia in the name would help with visibility of the underlying tech, but also tightly couples us to the Helia impl, pros and cons both ways. (PR for Helia) as far as |
I think that like normal IPFS gateways, consumers of the SW gateway are not necessarily going to be interested in the underlying tech (especially if it works well) which is why I think it's ok to remove Helia from the repo name. We absolutely should mention it on the gateway landing page with a link to the repo, but I don't think it's necessary for the repo name. |
I can get behind that thinking |
PSA: I will be moving to |
Reverting #202 as I finished ipfs/service-worker-gateway#80 and no longer need elevated permissions.
Reverting #202 as I finished ipfs/service-worker-gateway#80 and no longer need elevated permissions.
Reverting #202 as I finished ipfs/service-worker-gateway#80 and no longer need elevated permissions. Co-authored-by: ipfs-mgmt-read-write[bot] <104492829+ipfs-mgmt-read-write[bot]@users.noreply.github.com>
ipfs-shipyard org at github will be sunsetting later this year,
I propose moving to https://github.com/ipfs/service-worker-gateway
updateUCI_GITHUB_TOKEN
introduced in feat(ci): update inbrowser.link on release publish #117 after we finish move toipfs/
💭 we may want to switch to a bot one rather than using my PAT, so it does not look like I did PR: chore(main): release 1.0.0 #120cc @aschmahmann @SgtPooki @2color @achingbrain 👍 / 👎 ?
The text was updated successfully, but these errors were encountered: