Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork5.2k
Referenced oldest but still maintained version in contributing documenta...#2859
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
I'm going to close this one. The 2.2 branch wasn't deprecated when 2,2 was released (you can imagine why... ;-) ). And changing it in that branch is againstour update process:
|
From the document you are refering to:
For me, that's exactly the case with this PR. |
No, when 2.2 reached end of maintenance, branch 2.3 and higher are under maintenance, 2.2 is not. This PR is fixing it for the 2.2 branch, the 2.3+ branches are already updated with this. |
But we are talking about 2.1 which indeed reached its end of maintenance. Thus, according to the steps to be done pull requests for bugfixes should be based on the 2.2 branch. And this is currently not reflected in the documentation. If someone is willing to contribute he or she may currently be tempted to start with the 2.1 branch:
|
after a short chat with@xabbuh I think I was wrong. I'm a bit dizzy about all these versions currently, so I just reopen it and let@weaverryan decide on this one. |
Hi guys! Yes, it can be confusing, but I'm glad you all went back to our written documentation for our release process. That means we'll be consistent, and if there is some way to improve that process, we can change that document :). In this case, I also agree with@xabbuh's last comment - I think this PR perfectly addresses what should have happened when 2.1 reached end of life (we didn't have our release process in place then, so we're just catching up a little still, which is totally fine). So, I'm going to merge in - thanks Diego for this! |
Referenced oldest but still maintained version in contributing documenta...
...tion.