Skip to content

Commit 8cdf98c

Browse files
authored
Merge pull request #790 from ember-cli/release-plan
start using release-plan
2 parents ec487d1 + 6a99894 commit 8cdf98c

5 files changed

Lines changed: 663 additions & 1357 deletions

File tree

.github/workflows/plan-release.yml

Lines changed: 61 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,61 @@
1+
name: Plan Release
2+
on:
3+
workflow_dispatch:
4+
push:
5+
branches:
6+
- main
7+
- master
8+
pull_request_target: # This workflow has permissions on the repo, do NOT run code from PRs in this workflow. See https://securitylab.github.com/research/github-actions-preventing-pwn-requests/
9+
types:
10+
- labeled
11+
- unlabeled
12+
13+
concurrency:
14+
group: plan-release # only the latest one of these should ever be running
15+
cancel-in-progress: true
16+
17+
jobs:
18+
should-run-release-plan-prepare:
19+
name: Should we run release-plan prepare?
20+
runs-on: ubuntu-latest
21+
outputs:
22+
should-prepare: ${{ steps.should-prepare.outputs.should-prepare }}
23+
steps:
24+
- uses: release-plan/actions/should-prepare-release@v1
25+
with:
26+
ref: 'master'
27+
id: should-prepare
28+
29+
create-prepare-release-pr:
30+
name: Create Prepare Release PR
31+
runs-on: ubuntu-latest
32+
timeout-minutes: 5
33+
needs: should-run-release-plan-prepare
34+
permissions:
35+
contents: write
36+
issues: read
37+
pull-requests: write
38+
if: needs.should-run-release-plan-prepare.outputs.should-prepare == 'true'
39+
steps:
40+
- uses: release-plan/actions/prepare@v1
41+
name: Run release-plan prepare
42+
with:
43+
ref: 'master'
44+
env:
45+
GITHUB_AUTH: ${{ secrets.GITHUB_TOKEN }}
46+
id: explanation
47+
48+
- uses: peter-evans/create-pull-request@v8
49+
name: Create Prepare Release PR
50+
with:
51+
commit-message: "Prepare Release ${{ steps.explanation.outputs.new-version}} using 'release-plan'"
52+
labels: "internal"
53+
sign-commits: true
54+
branch: release-preview
55+
title: Prepare Release ${{ steps.explanation.outputs.new-version }}
56+
body: |
57+
This PR is a preview of the release that [release-plan](https://github.com/embroider-build/release-plan) has prepared. To release you should just merge this PR 👍
58+
59+
-----------------------------------------
60+
61+
${{ steps.explanation.outputs.text }}

.github/workflows/publish.yml

Lines changed: 30 additions & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -1,32 +1,41 @@
1-
name: Publish
1+
# For every push to the primary branch with .release-plan.json modified,
2+
# runs release-plan.
3+
4+
name: Publish Stable
25

36
on:
7+
workflow_dispatch:
48
push:
5-
tags:
6-
- '*'
9+
branches:
10+
- main
11+
- master
12+
paths:
13+
- '.release-plan.json'
14+
15+
concurrency:
16+
group: publish-${{ github.head_ref || github.ref }}
17+
cancel-in-progress: true
718

819
jobs:
920
publish:
10-
name: Publish to npm
11-
if: github.event_name == 'push' && startsWith(github.ref, 'refs/tags/v')
21+
name: "NPM Publish"
1222
runs-on: ubuntu-latest
13-
steps:
14-
- name: Checkout
15-
uses: actions/checkout@v3
23+
permissions:
24+
contents: write
25+
id-token: write
26+
attestations: write
1627

17-
- name: Install node
18-
uses: actions/setup-node@v3
28+
steps:
29+
- uses: actions/checkout@v6
30+
- uses: pnpm/action-setup@v4
31+
- uses: actions/setup-node@v6
1932
with:
20-
node-version: 16.x
33+
node-version: 22
2134
registry-url: 'https://registry.npmjs.org'
22-
23-
- name: install dependencies
24-
run: yarn install --frozen-lockfile --ignore-engines
25-
26-
- name: auto-dist-tag
27-
run: npx auto-dist-tag --write
28-
29-
- name: publish to npm
30-
run: npm publish
35+
cache: pnpm
36+
- run: npm install -g npm@latest # ensure that the globally installed npm is new enough to support OIDC
37+
- run: pnpm install --frozen-lockfile
38+
- name: Publish to NPM
39+
run: NPM_CONFIG_PROVENANCE=true pnpm release-plan publish
3140
env:
32-
NODE_AUTH_TOKEN: ${{ secrets.NPM_AUTH_TOKEN }}
41+
GITHUB_AUTH: ${{ secrets.GITHUB_TOKEN }}

RELEASE.md

Lines changed: 15 additions & 48 deletions
Original file line numberDiff line numberDiff line change
@@ -1,60 +1,27 @@
11
# Release Process
22

3-
Releases are mostly automated using
4-
[release-it](https://github.com/release-it/release-it/) and
5-
[lerna-changelog](https://github.com/lerna/lerna-changelog/).
3+
Releases in this repo are mostly automated using [release-plan](https://github.com/embroider-build/release-plan/). Once you label all your PRs correctly (see below) you will have an automatically generated PR that updates your CHANGELOG.md file and a `.release-plan.json` that is used to prepare the release once the PR is merged.
64

75
## Preparation
86

9-
Since the majority of the actual release process is automated, the primary
10-
remaining task prior to releasing is confirming that all pull requests that
11-
have been merged since the last release have been labeled with the appropriate
12-
`lerna-changelog` labels and the titles have been updated to ensure they
13-
represent something that would make sense to our users. Some great information
14-
on why this is important can be found at
15-
[keepachangelog.com](https://keepachangelog.com/en/1.0.0/), but the overall
16-
guiding principle here is that changelogs are for humans, not machines.
17-
18-
When reviewing merged PR's the labels to be used are:
19-
20-
* breaking - Used when the PR is considered a breaking change.
21-
* enhancement - Used when the PR adds a new feature or enhancement.
22-
* bug - Used when the PR fixes a bug included in a previous release.
23-
* documentation - Used when the PR adds or updates documentation.
24-
* internal - Used for internal changes that still require a mention in the
25-
changelog/release notes.
26-
27-
## Release
7+
Since the majority of the actual release process is automated, the remaining tasks before releasing are:
288

29-
Once the prep work is completed, the actual release is straight forward:
9+
- correctly labeling **all** pull requests that have been merged since the last release
10+
- updating pull request titles so they make sense to our users
3011

31-
* First, ensure that you have installed your projects dependencies:
32-
33-
```sh
34-
yarn install
35-
```
36-
37-
* Second, ensure that you have obtained a
38-
[GitHub personal access token][generate-token] with the `repo` scope (no
39-
other permissions are needed). Make sure the token is available as the
40-
`GITHUB_AUTH` environment variable.
41-
42-
For instance:
12+
Some great information on why this is important can be found at [keepachangelog.com](https://keepachangelog.com/en/1.1.0/), but the overall
13+
guiding principle here is that changelogs are for humans, not machines.
4314

44-
```bash
45-
export GITHUB_AUTH=abc123def456
46-
```
15+
When reviewing merged PR's the labels to be used are:
4716

48-
[generate-token]: https://github.com/settings/tokens/new?scopes=repo&description=GITHUB_AUTH+env+variable
17+
- breaking - Used when the PR is considered a breaking change.
18+
- enhancement - Used when the PR adds a new feature or enhancement.
19+
- bug - Used when the PR fixes a bug included in a previous release.
20+
- documentation - Used when the PR adds or updates documentation.
21+
- internal - Internal changes or things that don't fit in any other category.
4922

50-
* And last (but not least 😁) do your release.
23+
**Note:** `release-plan` requires that **all** PRs are labeled. If a PR doesn't fit in a category it's fine to label it as `internal`
5124

52-
```sh
53-
npx release-it
54-
```
25+
## Release
5526

56-
[release-it](https://github.com/release-it/release-it/) manages the actual
57-
release process. It will prompt you to to choose the version number after which
58-
you will have the chance to hand tweak the changelog to be used (for the
59-
`CHANGELOG.md` and GitHub release), then `release-it` continues on to tagging,
60-
pushing the tag and commits, etc.
27+
Once the prep work is completed, the actual release is straight forward: you just need to merge the open [Plan Release](https://github.com/ember-cli/ember-cli-htmlbars/pulls?q=is%3Apr+is%3Aopen+%22Prepare+Release%22+in%3Atitle) PR

package.json

Lines changed: 6 additions & 25 deletions
Original file line numberDiff line numberDiff line change
@@ -33,6 +33,9 @@
3333
"test:ember": "ember test",
3434
"test:node": "mocha node-tests/*.js"
3535
},
36+
"resolutions": {
37+
"ember-cli-htmlbars": "link:."
38+
},
3639
"dependencies": {
3740
"@ember/edition-utils": "^1.2.0",
3841
"babel-plugin-ember-template-compilation": "^2.0.0",
@@ -91,8 +94,7 @@
9194
"prettier": "^3.1.1",
9295
"qunit": "^2.20.0",
9396
"qunit-dom": "^2.0.0",
94-
"release-it": "^14.2.1",
95-
"release-it-lerna-changelog": "^3.1.0",
97+
"release-plan": "^0.17.4",
9698
"stylelint": "^15.11.0",
9799
"stylelint-config-standard": "^34.0.0",
98100
"stylelint-prettier": "^4.1.0",
@@ -102,6 +104,7 @@
102104
"@babel/core": ">= 7",
103105
"ember-source": ">= 4.0.0"
104106
},
107+
"packageManager": "pnpm@10.28.2",
105108
"engines": {
106109
"node": ">= 20"
107110
},
@@ -114,27 +117,5 @@
114117
"ember-addon": {
115118
"main": "lib/ember-addon-main.js",
116119
"configPath": "tests/dummy/config"
117-
},
118-
"resolutions": {
119-
"ember-cli-htmlbars": "link:."
120-
},
121-
"release-it": {
122-
"plugins": {
123-
"release-it-lerna-changelog": {
124-
"infile": "CHANGELOG.md",
125-
"launchEditor": true
126-
}
127-
},
128-
"git": {
129-
"tagName": "v${version}"
130-
},
131-
"github": {
132-
"release": true,
133-
"tokenRef": "GITHUB_AUTH"
134-
},
135-
"npm": {
136-
"publish": false
137-
}
138-
},
139-
"packageManager": "pnpm@10.28.2"
120+
}
140121
}

0 commit comments

Comments
 (0)