-
Notifications
You must be signed in to change notification settings - Fork 96
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
docs: Start on the Release Process #1319
base: dev
Are you sure you want to change the base?
docs: Start on the Release Process #1319
Conversation
fe690e2
to
e986e81
Compare
Hm, how about just a single numbered list instead of heading per step? Additional steps that come to mind:
|
e986e81
to
5d88460
Compare
Hmm. I don't care much. @karabowi added them in ChristianTackeGSI#1
Where would you suggest to put this?
done. |
That is usually, what I do in the context of a release among other things:
|
5d88460
to
466ea8b
Compare
466ea8b
to
8425225
Compare
@karabowi as you're currently in the process of creating a new release. Can you take another look at this? Maybe it would be good to merge and improve from there (if it's good enough for a merge)? If you want to take a look how the proposed changes look like when rendered: https://github.com/ChristianTackeGSI/FairRoot/blob/docs-contributing/CONTRIBUTING.md#user-content-creating-a-new-release |
8425225
to
9abcbff
Compare
WalkthroughWalkthroughThe Changes
Recent Review DetailsConfiguration used: CodeRabbit UI Files selected for processing (1)
Additional Context UsedLanguageTool (8)
Markdownlint (18)
Additional comments not posted (5)
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (invoked as PR comments)
Additionally, you can add CodeRabbit Configration File (
|
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.
Actionable comments posted: 4
Outside diff range and nitpick comments (7)
CONTRIBUTING.md (7)
Line range hint
15-15
: Consider using "first-time" instead of "first time" for grammatical correctness.
Line range hint
24-24
: Simplify "at this point in time" to "now" for conciseness.
Line range hint
45-45
: The phrase "will evolve over time" can be simplified to "will evolve" for conciseness.
Line range hint
65-65
: Use "and" after 'both' to correct the usage error in "both in commit messages as well as in PR comments".- both in commit messages as well as in PR comments if applicable. + both in commit messages and in PR comments if applicable.
Line range hint
76-77
: Move "Rarely" to a position after the subject for better readability and correct the verb form of "opt-out".- Rarely, it is needed to opt-out of *clang-format* for certain code sections + It is rarely needed to opt out of *clang-format* for certain code sections
Line range hint
53-64
: Correct the indentation of unordered list items to maintain consistency.- * regenerate/update existing images (cover new FairSoft releases? get - latest os updates to test closer to what a user may use) - * add new images (new os releases? which ship new compiler major versions - perhaps) - * remove old images - * For macs it involves similar steps, updating homebrew, perhaps the os - or the command line tools + * regenerate/update existing images (cover new FairSoft releases? get + latest os updates to test closer to what a user may use) + * add new images (new os releases? which ship new compiler major versions + perhaps) + * remove old images + * For macs it involves similar steps, updating homebrew, perhaps the os + or the command line tools
Line range hint
56-56
: Ensure headings are surrounded by blank lines for better readability and consistency.+ # Creating a new Release +
9abcbff
to
504fa1b
Compare
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.
Actionable comments posted: 2
Outside diff range and nitpick comments (2)
CONTRIBUTING.md (2)
Line range hint
24-24
: Consider simplifying the phrase "at this point in time" to "currently" for conciseness.
Line range hint
53-64
: Adjust the indentation of the unordered list items to match the expected levels.- * regenerate/update existing images (cover new FairSoft releases? get - latest os updates to test closer to what a user may use) - * add new images (new os releases? which ship new compiler major versions - perhaps) - * remove old images - * For macs it involves similar steps, updating homebrew, perhaps the os - or the command line tools + * regenerate/update existing images (cover new FairSoft releases? get + latest os updates to test closer to what a user may use) + * add new images (new os releases? which ship new compiler major versions + perhaps) + * remove old images + * For macs it involves similar steps, updating homebrew, perhaps the os + or the command line tools
504fa1b
to
2f0a5d4
Compare
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.
Actionable comments posted: 3
Outside diff range and nitpick comments (6)
CONTRIBUTING.md (6)
Line range hint
24-24
: Simplify the phrase for clarity.- These guidelines are horribly incomplete at this point in time, but one has to start somewhere ;) + These guidelines are still in development, but one has to start somewhere ;)
Line range hint
45-45
: Remove redundancy in the phrase.- The set of enabled checks will evolve over time. + The set of enabled checks will evolve.
Line range hint
65-65
: Correct the grammatical structure for clarity.- both in commit messages as well as in PR comments if applicable. + both in commit messages and in PR comments if applicable.
Line range hint
76-77
: Correct the verb form for "opt-out".- Rarely, it is needed to opt-out of *clang-format* for certain code sections + Rarely, it is needed to opt out of *clang-format* for certain code sections
Line range hint
53-53
: Standardize unordered list indentation.- * regenerate/update existing images (cover new FairSoft releases? get - latest os updates to test closer to what a user may use) - * add new images (new os releases? which ship new compiler major versions - perhaps) - * remove old images - * For macs it involves similar steps, updating homebrew, perhaps the os - or the command line tools + * regenerate/update existing images (cover new FairSoft releases? get + latest os updates to test closer to what a user may use) + * add new images (new os releases? which ship new compiler major versions + perhaps) + * remove old images + * For macs it involves similar steps, updating homebrew, perhaps the os + or the command line toolsAlso applies to: 57-57, 58-58, 61-61, 64-64, 132-132, 134-134, 136-136, 137-137
Line range hint
56-56
: Ensure headings are surrounded by blank lines for better readability.+ # Creating a new Release +
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.
Actionable comments posted: 5
Outside diff range and nitpick comments (4)
CONTRIBUTING.md (4)
Line range hint
24-24
: Simplify the phrase for clarity.- These guidelines are horribly incomplete at this point in time, but one has to start somewhere ;) + These guidelines are still incomplete, but one has to start somewhere ;)
Line range hint
45-45
: Remove redundancy in the phrase.- The set of enabled checks will evolve over time. + The set of enabled checks will evolve.
Line range hint
65-65
: Correct the conjunction usage for grammatical accuracy.- both in commit messages as well as in PR comments if applicable. + both in commit messages and in PR comments if applicable.
Line range hint
76-77
: Correct the verb form for "opt-out".- Rarely, it is needed to opt-out of *clang-format* for certain code sections + Rarely, it is needed to opt out of *clang-format* for certain code sections
We should be transparent about our release process. Co-Authored: @karabowi Co-Authored: @dennisklein
Added section about complicated patch branch merging process. Added section about major version.
6eaf8b4
to
a0f4847
Compare
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.
Actionable comments posted: 0
Outside diff range and nitpick comments (7)
CONTRIBUTING.md (7)
Line range hint
24-24
: Consider simplifying "at this point in time" to "currently" to enhance clarity.- These guidelines are horribly incomplete at this point in time, but one has to start somewhere ;) + These guidelines are horribly incomplete currently, but one has to start somewhere ;)
Line range hint
45-45
: The phrase "will evolve over time" can be simplified to "will evolve".- The set of enabled checks will evolve over time. + The set of enabled checks will evolve.
Line range hint
65-65
: Clarify the usage of "both" by adding "and".- both in commit messages as well as in PR comments if applicable. + both in commit messages and in PR comments if applicable.
Line range hint
76-77
: Correct the verb form for "opt-out".- Rarely, it is needed to opt-out of *clang-format* for certain code sections + Rarely, it is needed to opt out of *clang-format* for certain code sections
Line range hint
56-56
: Ensure that headings are surrounded by blank lines for proper Markdown formatting.- # Creating a new Release + + # Creating a new Release +
Line range hint
31-31
: Ensure that fenced code blocks are surrounded by blank lines for proper Markdown formatting.- ``` + + ``` +Also applies to: 81-81, 101-101
Line range hint
57-57
: Ensure that lists are surrounded by blank lines for proper Markdown formatting.- * Then I compare what is currently tested via CI and decide whether it + + * Then I compare what is currently tested via CI and decide whether it +
We should be transparent on our release process.
@dennisklein , @karabowi could you please help with this PR? Either add comments or add commits to the branch.
(I'll squash things before we declare this "ready for review".)
Rendered: https://github.com/ChristianTackeGSI/FairRoot/blob/docs-contributing/CONTRIBUTING.md#user-content-creating-a-new-release
Checklist:
dev
branchSummary by CodeRabbit
CONTRIBUTING.md
with detailed instructions for creating a new release, including steps for controlling status, checking CI coverage, creating commits, pushing patches and tags, releasing on GitHub, and merging tags onto the master branch.