-
Notifications
You must be signed in to change notification settings - Fork 2.7k
ci: add GitHub Action to check CHANGELOG consistency with PRs #12347
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
Conversation
'ci' | ||
]; | ||
|
||
const IGNORE_PRS = [ |
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.
refer: #12308 (comment)
ci/check_changelog_prs.ts
Outdated
} | ||
} | ||
|
||
main(); |
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.
await main()
?
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.
done.
name: Check Changelog | ||
|
||
on: | ||
push: |
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.
I'm wondering if we should just listen for changes in the CHANGEsLOG file?
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.
I'm wondering if we should just listen for changes in the CHANGEsLOG file?
we actually need to detect any missing changelog updates as early as possible when a PR is created. I just tested it and found that the push event does not trigger the CI when the PR is first submitted.
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.
seems fine to me
Description
This PR provides a script to check the changelog for completeness. If a PR is not of type docs, chore, test, or ci, and is not listed in the IGNORE_PRS variable, but is missing from the changelog, the script will raise an error.
Note: This script currently only checks changelog entries for versions after 3.8.0.
Test record:
Which issue(s) this PR fixes:
Checklist