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

Automate release notes generation #6187

Merged
merged 26 commits into from
Sep 16, 2021
Merged
Show file tree
Hide file tree
Changes from 12 commits
Commits
Show all changes
26 commits
Select commit Hold shift + click to select a range
5d2e6cf
Automate generation of release notes
pierlon May 8, 2021
499a6e8
Merge branch 'develop' into enhancement/5520-release-automation
pierlon Jun 27, 2021
162ee17
Limit release draft changelog generator to release branch being updated
pierlon Jun 27, 2021
69d92c1
Add `draft-release` GHA
pierlon Jun 28, 2021
05cca1f
Hide diff for `dist/index.js`
pierlon Jun 28, 2021
5819882
Fix syntax
pierlon Jun 29, 2021
fd2a0ad
Fix making release
pierlon Jun 28, 2021
81513c2
Don't hardcode repo name
pierlon Jun 29, 2021
d68ec22
Pass repo name to class
pierlon Jun 29, 2021
e52c598
Merge branch 'develop' of github.com:ampproject/amp-wp into enhanceme…
westonruter Jun 30, 2021
9613b0c
Add npm script to generate release changelog
pierlon Jun 30, 2021
e1eab3b
Ignore lint error
pierlon Jul 7, 2021
312f0d3
Parse arguments and validate them
pierlon Sep 2, 2021
5b6e433
Check release is a draft before updating it
pierlon Sep 2, 2021
752872d
Verify milestone and tag name match
pierlon Sep 2, 2021
36e0871
Merge branch 'develop' into enhancement/5520-release-automation
pierlon Sep 2, 2021
6f88a29
Update package-lock.json
pierlon Sep 2, 2021
237f3f8
Build `draft-release` GHA
pierlon Sep 2, 2021
e0dea20
Format JSDoc block
pierlon Sep 2, 2021
723c9f8
Merge branch 'develop' into enhancement/5520-release-automation
pierlon Sep 15, 2021
1a89432
Update package-lock.json
pierlon Sep 15, 2021
3efbadc
Merge branch 'develop' into enhancement/5520-release-automation
pierlon Sep 16, 2021
0c958d7
Update package-lock.json
pierlon Sep 16, 2021
d1ccd88
Update draft-release GHA binary
pierlon Sep 16, 2021
718b950
Suppress LGTM alert
pierlon Sep 16, 2021
3c87818
Try moving suppression to front of comment
pierlon Sep 16, 2021
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
23 changes: 23 additions & 0 deletions .github/actions/draft-release/.eslintrc
Original file line number Diff line number Diff line change
@@ -0,0 +1,23 @@
{
"root": true,
"extends": [
"airbnb",
"prettier"
],
"env": {
"es6": true,
"node": true
},
"rules": {
"prettier/prettier": [
"error",
{
"singleQuote": true,
"printWidth": 120
}
]
},
"plugins": [
"prettier"
]
}
2 changes: 2 additions & 0 deletions .github/actions/draft-release/.gitattributes
Original file line number Diff line number Diff line change
@@ -0,0 +1,2 @@
# Mark generated files so diffs are hidden by default.
dist/index.js linguist-generated=true
25 changes: 25 additions & 0 deletions .github/actions/draft-release/README.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,25 @@
# Draft release action

This action drafts a release for the specified milestone and release branch.

## Inputs

### `milestone`

**Required** Milestone name.

## Outputs

### `asset_upload_url`

The URL for uploading assets to the release.

## Example usage

```yaml
uses: ./.github/actions/draft-release
env:
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
with:
milestone: v3.2.1
```
15 changes: 15 additions & 0 deletions .github/actions/draft-release/action.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,15 @@
name: 'Draft release'
description: 'Drafts a release based on the specified milestone'
inputs:
milestone:
description: 'Milestone name'
required: true
release_branch:
description: 'Release branch name'
required: true
outputs:
asset_upload_url:
description: 'The URL for uploading assets to the release'
runs:
using: 'node12'
main: 'dist/index.js'
Copy link
Member

Choose a reason for hiding this comment

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

Where does this file come from? Forgive my ignorance.

Copy link
Member

Choose a reason for hiding this comment

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

Oh, nevermind. It comes from npm run build for this sub-package.

Loading