jupyter-server / jupyter_ydoc

Jupyter document structures for collaborative editing using Yjs/pycrdt
https://jupyter-ydoc.readthedocs.io
BSD 3-Clause "New" or "Revised" License
25 stars 16 forks source link

Cannot make a release #242

Closed davidbrochart closed 1 week ago

davidbrochart commented 2 weeks ago

Description

I created a draft release but I cannot publish it. There is an error with npm whoami, maybe I don't have the rights?

cc @Zsailer @fcollonval @krassowski @jtpio

jtpio commented 2 weeks ago

@davidbrochart I bumped your rights to "Admin". They were set to "Write" reviously, and with the "mixed roles" warning.

Would you be able to try again?

jtpio commented 2 weeks ago

Also it looks like the draft release was created with @fcollonval account: https://github.com/jupyter-server/jupyter_ydoc/releases/tag/untagged-0497b4902adcd4680709

Maybe we should streamline the process for this repo and use the GitHub app for the jupyter-server organization, like for the other repos under this org.

davidbrochart commented 2 weeks ago

Would you be able to try again?

Thanks @jtpio, but it doesn't seem to be enough, see here.

Maybe we should streamline the process for this repo and use the GitHub app for the jupyter-server organization, like for the other repos under this org.

Probably, but I'm not sure I have the rights for this either?

jtpio commented 2 weeks ago

OK it looks like I should be able to add the proper credentials from the Jupyter password to this repo.

I opened https://github.com/jupyter-server/jupyter_ydoc/pull/244 to update the workflows.

jtpio commented 2 weeks ago

I just set the two variables for the release environment on this repo:

image

And invited jupyter-server-release-bot to join the jupyter org on npm.

And also just added the trusted publisher to PyPI:

image

jtpio commented 1 week ago

@davidbrochart would you like to try running the workflows again? I deleted the previous draft release.

davidbrochart commented 1 week ago

Thanks, it failed again but with a different error:

***github.com/jupyter-server/jupyter_ydoc.git
remote: error: GH006: Protected branch update failed for refs/heads/main.
remote: error: Changes must be made through a pull request.
To https://github.com/jupyter-server/jupyter_ydoc.git
 * [new tag]         @jupyter/ydoc@3.0.0-a1 -> @jupyter/ydoc@3.0.0-a1
 * [new tag]         v3.0.0a1 -> v3.0.0a1
 ! [remote rejected] HEAD -> main (protected branch hook declined)
error: failed to push some refs to 'https://github.com/jupyter-server/jupyter_ydoc.git'
jtpio commented 1 week ago

Looks like it's because there is already a tag on the repo?

Maybe worth trying again by deleting the draft release and the tags, and running the workflows again?

davidbrochart commented 1 week ago

That's what I did, but I'll try again.

davidbrochart commented 1 week ago

Same result, maybe this commit should be reverted?

jtpio commented 1 week ago

Ah yes, let's try to revert that commit and publish 3.0.0a0 (not 3.0.0a1)

davidbrochart commented 1 week ago

Unfortunately it still doesn't work (see here).

jtpio commented 1 week ago

OK it might be related to some kind of branch protection rule, let me check.

jtpio commented 1 week ago

I just added the same ruleset as for the Jupyter Server repository: https://github.com/jupyter-server/jupyter_server

And restarted the workflow, let's see if that fixes it.

jtpio commented 1 week ago

OK looking good this time: https://github.com/jupyter-server/jupyter_ydoc/releases/tag/v3.0.0a0

Closing as fixed!

davidbrochart commented 1 week ago

Thanks so much @jtpio.