-
Notifications
You must be signed in to change notification settings - Fork 26
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
Add action for checking changelog updates #1358
Conversation
runs-on: ubuntu-latest | ||
steps: | ||
- uses: actions/checkout@v4 | ||
- if: ${{ !contains(github.event.pull_request.labels.*.name, 'skip_changelog') }} |
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.
Adding filter for changed files could save time here
- name: Filter changes
id: changes
uses: dorny/paths-filter@v3
with:
filters: |
has_changes:
- 'desc/**'
For the changelog, we don't want to include the changes in other folders anyway.
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.
this whole action takes a few seconds so don't think its really worth it.
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.
By time, I mean adding the flag. If this test runs for every PR and fails, then that small red cross could be annoying :)
Adds a GH action that ensures
CHANGELOG.md
is updated with each PR, unless the PR is labelledskip_changelog
Resolves #1340