-
Notifications
You must be signed in to change notification settings - Fork 4.8k
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
chore(changelog): make it more usable #11519
Conversation
`make -C CHANGELOG` to use it. KAG-2527
repo's naming conventions
echo | ||
|
||
echo "What is the type of the change (feature, bugfix, dependency, deprecation, breaking_change, performance)?" | ||
select type in feature bugfix dependency deprecation breaking_change performance; do |
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.
The enum of type
and scope
are defined in the CHANGELOG/schema.json
, Do you think removing hard code(feature|bugfix|dependency|deprecation|breaking_change|performance) here would make things more consistent?
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.
Temporary measure, let's discuss later. We need something more reasonable immediately.
id: changelog-check | ||
uses: tj-actions/changed-files@2f7246cb26e8bb6709b6cbfc1fec7febfe82e96a # v37 | ||
with: | ||
files: 'CHANGELOG/unreleased/**/*.yaml' |
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.
Since we have open PRs using the .yaml extension instead of yml, could we support both file extensions for a while (until everyone is using the new system)? Then we can rename everything to .yml and stop supporting yaml.
Superseded by #11652 |
No description provided.