Skip to content

Conversation

@M-Kusumgar
Copy link
Contributor

No description provided.

@david-mears-2
Copy link
Contributor

david-mears-2 commented Dec 22, 2025

I merged a package version update from 1.1.3 to 1.1.4 into main via #56, but then when I tried to publish the new version to npm from main, I got

403 Forbidden - PUT https://registry.npmjs.org/@reside-ic%2fskadi-chart - You cannot publish over the previously published versions: 1.1.4

so you must have published from this branch already. Now we have two conflicting ideas in the git history of what 1.1.4 refers to.

I'm not sure how best to prevent this happening in the future - I feel like we prefer to relax the norm that 'you can only publish versions from main' so that wodin or vaxviz can build off specific WIP branches. We could say that WIP versions published for that purpose should use a pre-release version number like 1.0.0-alpha or 1.0.0-alpha.1 or 1.0.0-beta, with the flag npm publish --tag alpha or npm publish --tag beta.

In any case, it would probably help if we always communicated when we are about to publish a new version to npm.

{
"name": "@reside-ic/skadi-chart",
"version": "1.1.3",
"version": "1.1.4",
Copy link
Contributor

Choose a reason for hiding this comment

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

just leaving a comment to remind to make sure this increments from main, or at least does not decrement

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants