-
Notifications
You must be signed in to change notification settings - Fork 0
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
feat: adds lint-release-notes action #20
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just some minor comments
semantic-release/src/index.ts
Outdated
@@ -21,6 +21,8 @@ const OUTPUTS = { | |||
new_release_major_version: "new-release-major-version", | |||
new_release_minor_version: "new-release-minor-version", | |||
new_release_patch_version: "new-release-patch-version", | |||
last_release_version: "last-release-version", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sort these alphabetically (not sure how the linter didn't pick this)
semantic-release/action.yaml
Outdated
@@ -43,6 +43,12 @@ outputs: | |||
new-release-patch-version: | |||
description: "Patch version of the new release" | |||
value: ${{ steps.semantic-release.outputs.new-release-patch-version }} | |||
last-release-version: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sort these alphabetically
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Update the readme adding these to it (we haven't automated this - not sure if there is a way)
semantic-release/src/index.ts
Outdated
if (nextRelease) { | ||
setOutput(OUTPUTS.last_release_version, lastRelease.version); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Update the tests to check for these outputs
008a92c
to
def2346
Compare
Pull Request Test Coverage Report for Build 6193920728
💛 - Coveralls |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
+1 to updating the README and sorting params. Looks good otherwise
4809008
to
504725d
Compare
It seems the breaking changes comment shouldn't have been created though |
e1c21b7
to
875d39c
Compare
@tagoro9 it will break until it can self reference the updated version of semantic release
Maybe there's a better way to do this. I'd copied how it was being done elsewhere. But it was good to pull the PRs apart. |
67dfbd0
to
21b69b4
Compare
3310c0b
to
adbcde9
Compare
This adds a new check that enforces that a breaking changes document has been created.
adbcde9
to
04e002a
Compare
🎉 This PR is included in version 2.3.0 🎉 The release is available on:
Your semantic-release bot 📦🚀 |
Description
lint-release-notes
action that ensures that breaking change docs have been created and provides a preview of the release notes.actions-release/release-notes-preview
🚀 PR created with fotingo