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

FIX Ensure changelog passes linting #249

Conversation

GuySartorelli
Copy link
Member

Tested this with a dummy run and it works well.
Some commit messages may still introduce unexpected linting issues - we can look at whether the current set of rule exclusions is appropriate if/when that happens, but for now those rules are the only ones that need excluding. I don't want to exclude linting altogether for them in case some commit message has something which would result in poorly rendered markdown.

Issue

@GuySartorelli GuySartorelli marked this pull request as draft November 9, 2023 23:44
@GuySartorelli GuySartorelli force-pushed the pulls/master/lint-friendly-changelog branch from aa6f5c9 to 9185510 Compare November 10, 2023 00:08
@GuySartorelli GuySartorelli marked this pull request as ready for review November 10, 2023 00:08
@GuySartorelli GuySartorelli force-pushed the pulls/master/lint-friendly-changelog branch from 9185510 to 87ef34b Compare November 10, 2023 00:11
@GuySartorelli GuySartorelli force-pushed the pulls/master/lint-friendly-changelog branch from 87ef34b to 3de09ba Compare November 10, 2023 00:14
@emteknetnz emteknetnz merged commit 45d8d1a into silverstripe:master Nov 13, 2023
9 checks passed
@emteknetnz emteknetnz deleted the pulls/master/lint-friendly-changelog branch November 13, 2023 00:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants