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

[BUG] Publish doesn't work for a renamed package #63

Closed
3 tasks
ndr-brt opened this issue Jan 20, 2023 · 8 comments
Closed
3 tasks

[BUG] Publish doesn't work for a renamed package #63

ndr-brt opened this issue Jan 20, 2023 · 8 comments
Labels
bug Something isn't working

Comments

@ndr-brt
Copy link

ndr-brt commented Jan 20, 2023

Is this Bug Present in the upstream API Server?

  • Yes
  • No
  • Unsure

What is the Bug
The url of our plugin changed (from atom-* to pulsar-*) and the publication started to fail again.

How to Replicate the Bug
After login, running pulsar -p publish patch caused:

Preparing and tagging a new version ✓
Pushing v3.16.44 tag ✓
Publishing tidalcycles@v3.16.44 ✗
Creating new version failed: The package.json at owner/repo isn't valid
@ndr-brt ndr-brt added the bug Something isn't working label Jan 20, 2023
@confused-Techie
Copy link
Member

Thanks a ton for the bug report, I really appreciate the information, and will look into this issue.

@ndr-brt
Copy link
Author

ndr-brt commented Jan 23, 2023

If you need a retry please ping me on discord ;)

@confused-Techie
Copy link
Member

If you need a retry please ping me on discord ;)

I will absolutely let you know

@ndr-brt
Copy link
Author

ndr-brt commented Feb 11, 2023

@confused-Techie there could be the possibility to delete the tidalcycles packages and upload it again? Would that work?
Because now the need to update it is becoming more and more necessary.

@confused-Techie
Copy link
Member

@confused-Techie there could be the possibility to delete the tidalcycles packages and upload it again? Would that work? Because now the need to update it is becoming more and more necessary.

You could do that, although the name tidalcycles would not be reusable by default.
We could try to make an exception and have manual intervention for the name to become available again, but if you'd like, we have merged all the changes with what I believe will fix this issue after #50

Now I'm just trying to clean up a few things to give us more faith in that new system. But I can try to sprint these changes this weekend and get them pushed to prod. But let me know what you'd like to do and I'll work with you

@ndr-brt
Copy link
Author

ndr-brt commented Feb 14, 2023

Great to hear that, I'll wait to see if the issue will be fixed by the next rollout 🙌

@confused-Techie
Copy link
Member

@ndr-brt If possible would you be willing to give this another try?

We now have success with users publishing new package versions.

Keep in mind the new pinned issue on this repo. As in if it says you successfully published the new version, keep in mind the website and Pulsar might not show it, but using the API you'll be able to see the new version, or if you have the package installed you'll be prompted for the upgrade to your newer version.

Otherwise using ppm upgrade --list will show the newer update

@ndr-brt
Copy link
Author

ndr-brt commented Feb 21, 2023

@confused-Techie it worked! no errors 🥳 thanks a lot. I will close the issue

@ndr-brt ndr-brt closed this as completed Feb 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants