-
-
Notifications
You must be signed in to change notification settings - Fork 16.7k
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
Update release process documentation #5513
base: master
Are you sure you want to change the base?
Conversation
344da98
to
060d8da
Compare
Great work, but I am not really convinced that the Express way / Node.js way is the good one here. A lot less maintenance, a lot less impact and manual workflow
We can discuss that when you want, but for me this would be simpler, and it would be the same process for all packages (not just express) |
We actually do it on the step 7 (partially). If we work directly with the major branches we depend a lot on what is included there in terms of changes (minor, patch...) and it will be hard to select what we ship and when (specially thinking on v5/v6/v7 when we only have partial features, etc... Also this can be challenging if we want to do just a security release fast (patching) as we will need to create more ad-hoc branches, etc...
Also I don't expect this process to be the same for other libraries in the project. |
that is my main issue, I am not seeing why this is not the same process. For me we should be able to deliver things quickly, like other framework that have multiple releases per months. After all, in my eyes, Express is just "another library" so why not follow a more traditional /simplified way? (This process would be for the v5 - maybe not v4) I am not seeing that we should create a new version every day but even if we have a security update, then let's push what's already merged (that was reviewed and validated, so it should be working) We could even say "Hey, every Monday, if there is something new we do a release if not let's see in a week". This way we have small and quick incremental changes. |
Have y’all consider using the semantic-release tooling? I’m using it in Hubot. |
|
||
### Non-patch flow | ||
You should notify the community of your | ||
intent to release. This can be done by creating a message in the slack channel |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should there be a link to the Slack account/channel?
I think I agree with @sheplu on this, but I have to say seeing it in this form (as a pr changing existing docs) makes it REALLY hard to fully grok the differences. IMO this conversation would be a lot more clear it we had a plain issue write up of the proposal before a PR version. Because I am not sure we really have clarity on the differences and what the ideal even is, I feel like we should start with a new issue and outline 2 or 3 options for workflows we can compare. |
thanks for all the feedback! I will work in an issue to present this version and a simpler one, so we can have a better high level discussion. Also I need to include an step to update the website: (Ref: expressjs/expressjs.com#1469) |
This is a new version for the documentation. It is based on the experience doing #5505 but with some tweaks.
Key assumptions
master
branch and pick the relevant commits once doing a releaseSome notes
4.x
and5.x
in order to make this guide accurate (if you try to follow it in a fork o similar)