docs(release): add release process guide for internal contributors #19543
+80
−0
Conversation
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
jrainville
requested review from
a team and
iurimatias
and removed request for
a team
Member
Member
Member
Member
✔️ status-desktop/e2e/prs-windowsPR19543 🔹 ~24 min 🔹 3729a85 🔹 📦 tests/e2e-windows package |
noeliaSD
approved these changes
Dec 10, 2025
Contributor
noeliaSD
left a comment
noeliaSD
left a comment
There was a problem hiding this comment.
Thanks for the update! I’ve added one comment, and everything else LGTM.
docs/internal/release-process.md
Outdated
|
|
||
| 9. Fixes for issues identified during the RC phase **must** be worked on and committed on the release branch **first**. | ||
|
|
||
| 10. Once a fix has been applied to the release branch, it **must** be ported to the master branch. |
Contributor
There was a problem hiding this comment.
I would also clarify who is responsible for this step. Should the same developer who merged the commits into the release branch create the corresponding cherry-pick PR to master once it’s integrated? Or will there be a designated person during this phase to review everything added to the release and ensure it is properly ported back to master?
Member
Member
✔️ status-desktop/prs/linux/x86_64/tests-ui/PR-19543#2 🔹 ~13 min 🔹 7bb8695 🔹 📦 tests/ui package |
Member
Member
Member
Member
jrainville
force-pushed
the
docs/release-process-guide
branch
from
7bb8695 to
6204fc0
Compare
Member
✔️ status-desktop/prs/android/arm64/package/PR-19543#3 🔹 ~9 min 26 sec 🔹 6204fc0 🔹 📦 android/arm64 package |
Member
Member
✔️ status-desktop/prs/linux/x86_64/tests-ui/PR-19543#3 🔹 ~15 min 🔹 6204fc0 🔹 📦 tests/ui package |
Member
✔️ status-desktop/e2e/prs-windowsPR19543 🔹 ~24 min 🔹 7bb8695 🔹 📦 tests/e2e-windows package |
Member
Member
Member
Member
Member
✔️ status-desktop/e2e/prspr19543 🔹 ~15 min 🔹 6204fc0 🔹 📦 tests/e2e package |
Member
✔️ status-desktop/e2e/prs-windowsPR19543 🔹 ~22 min 🔹 6204fc0 🔹 📦 tests/e2e-windows package |
Member
Jenkins BuildsClick to see older builds (42)
|
jrainville
requested review from
a team,
Khushboo-dev-cpp,
alaibe and
igor-sirotin
and removed request for
a team
caybro
reviewed
Dec 17, 2025
docs/internal/release-process.md
Outdated
| 3. Only bug fixes **must** be provided to the release branch. | ||
|
|
||
| 4. Moreover, only **critical** bug fixes **should** be added to the [Release Board](https://github.com/orgs/status-im/projects/97/views/1) and committed to the release branch. | ||
| 1. Critical bugs are issues that affect the **behaviour** of the features. |
Member
There was a problem hiding this comment.
I'd argue critical bugs are first and foremost issues that affect:
- security
- potential data or funds loss
- crashes
Everything else is "severe" at best imho :)
| 10. Once a fix has been applied to the release branch, it **must** be ported to the master branch by the author of the commit. | ||
|
|
||
| 11. A new RC build **can** be triggered every day, if there are new fixes in the release branch. | ||
|
|
Member
There was a problem hiding this comment.
I'm missing a short note about how we tag and version the releases
docs/internal/release-process.md
Outdated
|
|
||
| The remaining features listed on the [Roadmap](https://github.com/status-im/status-app/blob/master/docs/roadmap.md) on the same milestone, but that were not identified as **key**, will simply be pushed to the next milestone. | ||
|
|
||
| All new features **must** implement a **feature flag**. Therefore, unfinished features **must not** affect master and the release. |
Member
There was a problem hiding this comment.
I'd say unfinished features are fine in master?
Member
Author
There was a problem hiding this comment.
Yeah, I guess I could refine the wording. I meant that if a feature is unfinished, it shouldn't make master break or anything severe, because the feature flag would prevent that.
One could argue that master could be temporarily broken, but we should strive for that to never happen.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
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.
Adds a new internal guide called "Release Process Guide" detailing how released ought to be done.
Please review and give any feedback