-
-
Notifications
You must be signed in to change notification settings - Fork 5.2k
The Documentation Release Process #2720
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
Merged
Merged
Changes from all commits
Commits
Show all changes
2 commits
Select commit
Hold shift + click to select a range
File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
I'd like to add: except if the issues the PR fixes are milestoned with a specific version
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.
But if an issue is milestoned to, say, 2.0, then it either wouldn't be fixed (since we're passed maintenance), or it should only be fixed for branches that are still under maintenance. What do you think?
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.
Let's take a milestoned issue: #2719 In 2.2, the routing system has changed and we use a lot of the deprecated methods in the documentation. Imagine the 2.2 branch was deprecated, isn't it completly wrong to not update that version but only update it in 2.3, as if the change was made in Symfony 2.3? We then have not working code in our documentation, just because we kept strict to the deprecation rule.
I think you can start deprecating things completely after all big things are fixed/documented. You can't deprecated the 2.1 version of the sf core if you know that it has some big bugs.
While we have a great doc team, we can't fully keep up with the core code (as that team is still much bigger). That's why I think we should not fully deprecate a doc branch, until all milestoned issues are fixed. Once a branch is fully deprecated, it needs to be fixed on the oldest maintained branch.
BTW, I recently removed the wrong milestones from issues, as some issues were milestoned but didn't really belong to a specific version,
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.
Hey Wouter!
I'm going to merge this in today, along with all of the other related PRs for versioning stuff and any other changes we need. Your situation here makes sense, but I'm not going to include it, at least for now. Of course, in reality, if we find some critical issue like this, we can bend our own rules. If a version has reached end-of-life, then it means it was released at least 1 year earlier, and we should certainly be able to get all of the critical fixes done in that time. Really, we should be able to get things updated during the "stabilization period.
Not including it here keeps the rule simple to read, though of course we can bend it if any rare item comes up.
Thanks!
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.
ok, I'm fine with merging this @weaverryan
(in case you are going to merge a lot of of PRs today, I'll update my open PRs within 3 hours to make them mergable)