Commit 5a588ff
doc: add updating expected assets to release guide
Add a section to the Major Releases section of the release guide
to cover updating the list of expected files for the promotion tool.
PR-URL: #40470
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Beth Griggs <bgriggs@redhat.com>1 parent 7cee125 commit 5a588ff
1 file changed
Lines changed: 11 additions & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -900,6 +900,15 @@ test, or doc-related are to be listed as notable changes. Some SEMVER-MINOR | |||
| 900 | 900 | commits may be listed as notable changes on a case-by-case basis. Use your | |
| 901 | 901 | judgment there. | |
| 902 | 902 | ||
| 903 | + ### Update the expected assets | ||
| 904 | + | ||
| 905 | + The promotion script does a basic check that the expected files are present. | ||
| 906 | + Open a pull request in the Build repository to add the list of expected files | ||
| 907 | + for the new release line as a new file, `v{N}.x` (where `{N}` is the major | ||
| 908 | + version of the release), in the [expected assets][] folder. The change will | ||
| 909 | + need to be deployed onto the web server by a member of the [build-infra team][] | ||
| 910 | + before the release is promoted. | ||
| 911 | + | ||
| 903 | 912 | ### Snap | |
| 904 | 913 | ||
| 905 | 914 | The Node.js [Snap][] package has a "default" for installs where the user hasn't | |
@@ -914,5 +923,7 @@ take place once a new LTS line has been released. | |||
| 914 | 923 | [Node.js Snap management repository]: https://github.com/nodejs/snap | |
| 915 | 924 | [Partner Communities]: https://github.com/nodejs/community-committee/blob/HEAD/governance/PARTNER_COMMUNITIES.md | |
| 916 | 925 | [Snap]: https://snapcraft.io/node | |
| 926 | + [build-infra team]: https://github.com/orgs/nodejs/teams/build-infra | ||
| 927 | + [expected assets]: https://github.com/nodejs/build/tree/HEAD/ansible/www-standalone/tools/promote/expected_assets | ||
| 917 | 928 | [nodejs.org release-post.js script]: https://github.com/nodejs/nodejs.org/blob/HEAD/scripts/release-post.js | |
| 918 | 929 | [nodejs.org repository]: https://github.com/nodejs/nodejs.org | |
0 commit comments