-
Notifications
You must be signed in to change notification settings - Fork 703
[docs] update review.md with details about backports #20142
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
Conversation
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.
Pull Request Overview
This PR enhances the documentation review guide by distinguishing review steps for new main-branch content and backported changes.
- Adds a placeholder section for main-branch PR guidance
- Introduces a detailed backport review checklist
- Clarifies branching and publication process for stable-branch backports
Comments suppressed due to low confidence (2)
ydb/docs/en/core/contributor/documentation/review.md:57
- This section is currently empty. Either populate it with the expected review steps for new main-branch PRs or remove the placeholder heading.
## If the PR is new content for the main branch
ydb/docs/en/core/contributor/documentation/review.md:57
- The heading level here matches the top-level checklist heading. Consider using a third-level heading (
###
) to nest it under the main Checklist section for better structure.
## If the PR is new content for the main branch
🟢 |
⚪ Test history | Ya make output | Test bloat
🟢 |
⚪ Test history | Ya make output | Test bloat
🟢 |
🔄 New commits pushed — @anton-bobkov please take a look. |
⚪ Test history | Ya make output | Test bloat
🟢 |
⚪ Test history | Ya make output | Test bloat
🟢 |
✅ Documentation buildRevision built successfully |
🔄 New commits pushed — @anton-bobkov please take a look. |
⚪ Test history | Ya make output | Test bloat
🟢 |
⚪ Test history | Ya make output | Test bloat
🟢 |
✅ Documentation buildRevision built successfully |
Changelog entry
...
Changelog category
Description for reviewers
...