Skip to content
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

Document the release process #7412

Merged
merged 9 commits into from
Feb 24, 2023
Merged

Conversation

edolstra
Copy link
Member

@edolstra edolstra commented Dec 6, 2022

Closes #3679

Copy link
Member

@thufschmitt thufschmitt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's great to have this documented!

Just left a few comments, but that's good overall

maintainers/release-process.md Outdated Show resolved Hide resolved
maintainers/release-process.md Outdated Show resolved Hide resolved
maintainers/release-process.md Outdated Show resolved Hide resolved
maintainers/release-process.md Outdated Show resolved Hide resolved
maintainers/release-process.md Show resolved Hide resolved
edolstra and others added 4 commits December 6, 2022 15:26
Co-authored-by: Théophane Hufschmitt <7226587+thufschmitt@users.noreply.github.com>
Co-authored-by: Théophane Hufschmitt <7226587+thufschmitt@users.noreply.github.com>
maintainers/release-process.md Outdated Show resolved Hide resolved
maintainers/release-process.md Outdated Show resolved Hide resolved
Co-authored-by: Théophane Hufschmitt <7226587+thufschmitt@users.noreply.github.com>
Co-authored-by: Robert Hensing <roberth@users.noreply.github.com>
maintainers/release-process.md Outdated Show resolved Hide resolved
maintainers/release-process.md Outdated Show resolved Hide resolved
maintainers/release-process.md Outdated Show resolved Hide resolved
* In a checkout of the Nix repo, make sure you're on `master` and do a
`git pull`.

* Move the contents of `doc/manual/src/release-notes/rl-next.md`
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is important: Please link to all referenced files with their GitHub URL, such that they will be easier to find.

While no one except those who know how it works will actually use these links to practice the release process, it will help potential contributors to orient in the code.

maintainers/release-process.md Outdated Show resolved Hide resolved
maintainers/release-process.md Outdated Show resolved Hide resolved
maintainers/release-process.md Outdated Show resolved Hide resolved
maintainers/release-process.md Outdated Show resolved Hide resolved
maintainers/release-process.md Outdated Show resolved Hide resolved
@nixos-discourse
Copy link

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:

https://discourse.nixos.org/t/tweag-nix-dev-update-41/23848/1

Co-authored-by: Valentin Gagarin <valentin.gagarin@tweag.io>
Copy link
Contributor

@blaggacao blaggacao left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LG(enough)TM, could probably be merged?

@nixos-discourse
Copy link

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:

https://discourse.nixos.org/t/nix-team-report-2022-10-2023-03/27486/1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

The release process is not clear
6 participants