| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
Sorry, something went wrong.
|
This looks much better 👌 |
Sorry, something went wrong.
| @@ -1,3 +1,3 @@ | |||
| { | |||
| ".": "6.26.1" | |||
There was a problem hiding this comment.
Ok, this looks like a better PR, but for all the work we've done and the significant dependency updates, I think this should be a minor, if not major, release after 3 years. @jdalrymple any thoughts on this? I can read through the docs again to understand how we ensure that at least 6.27 is the new version.
Sorry, something went wrong.
There was a problem hiding this comment.
One way is to merge a commit with
Release-As: 6.27.0
On the last line and it'll override the autodetection
Sorry, something went wrong.
There was a problem hiding this comment.
OH - that's amazing. I'll try that @michaelfaith I am testing a release candidate / pre release now to test out trusted publishing. i'll try to push an empty commit with that message.
Sorry, something went wrong.
There was a problem hiding this comment.
It worked perfectly!! i'll update our docs!
Sorry, something went wrong.
There was a problem hiding this comment.
It would def say major
Sorry, something went wrong.
|
Ok, maybe 7.x doesn't make sense here, but at least 6.27 does, given how long it's been since the last release and the significant dependency updates that could warrant breaking changes in a user's environment. I'm not sure, but 6.27 seems reasonable. Even looking at the commit history, it wouldn't follow a typical semantic minor bump. |
Sorry, something went wrong.
It does!! Thank you so much for noticing this. I need to look at which commit types would trigger a minor vs. major release. What is unclear here is that it's been ~3 years since a release was made! And we made some significant updates to dependencies. But it's not breaking for users because the front-end API hasn't changed, only the backend. |
Sorry, something went wrong.
feat! = major |
Sorry, something went wrong.
Wouldnt i be breaking simply by the fact that the nodejs versions supported has changed? Also our switch to pure ESM vs commonjs will break any consumers who are commonjs only |
Sorry, something went wrong.
That's a good point. So we should have created a fix!: for the commit around the ESM to CJS syntax as that would trigger a major bump? Thanks to Michael, it's very easy to push an empty commit that bumps the version, so we just need to decide on the most appropriate version. I am very close to getting the release to work, so I'm totally ok with deleting and creating a new one 😆 I've only done it like 6 times so far. I didn't actually know that the ESM migration would break consumers workflows using pure CJS so that does seem fair to me. |
Sorry, something went wrong.
Yea that would have made sense - my bad for not catching that when we did it |
Sorry, something went wrong.
| Back | FazBrowse Home | New Git URL |
🤖 I have created a release beep boop
6.27.0 (2026-08-07)
🔥 Bug Fixes
📚 Documentation
⚙️ Chores
⌨️ Code Refactoring
🧪 Testing
🛠️ Build System
This PR was generated with Release Please. See documentation.