Skip to content

chore: changelog check#565

Closed
joergmann wants to merge 5 commits intomainfrom
test/changelogCheck
Closed

chore: changelog check#565
joergmann wants to merge 5 commits intomainfrom
test/changelogCheck

Conversation

@joergmann
Copy link
Copy Markdown
Contributor

@joergmann joergmann commented Aug 8, 2025

joergmann and others added 4 commits August 8, 2025 10:52
…in permissions

Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
@joergmann joergmann changed the title Test/changelog check fix: changelog check Aug 8, 2025
@daogrady daogrady added the skip changelog Do not require an entry in the changelog label Aug 11, 2025
@daogrady
Copy link
Copy Markdown
Contributor

Thanks for this contribution!
Do we still want to be able to skip the workflow, i.e. by adding a label? This wf does not seem to react to the skip workflow label as of now.
Seeing that this particular PR fails, I guess chore: or ci: would be the more fitting prefix. Or did you choose fix: on purpose to demonstrate the workflow?

8ydfc

@joergmann
Copy link
Copy Markdown
Contributor Author

Thanks for this contribution! Do we still want to be able to skip the workflow, i.e. by adding a label? This wf does not seem to react to the skip workflow label as of now.

Do we need the label at all? I think the purpose of a PR should be reflected in the conventional commit type.

Seeing that this particular PR fails, I guess chore: or ci: would be the more fitting prefix. Or did you choose fix: on purpose to demonstrate the workflow?

You're right, this PR is just for testing.

@daogrady
Copy link
Copy Markdown
Contributor

Do we need the label at all? I think the purpose of a PR should be reflected in the conventional commit type.

Fully agree! I just kinda like having an "emergency" mechanism, but I guess I can always just bypass the rule when merging. 👍

@joergmann joergmann changed the title fix: changelog check chore: changelog check Aug 18, 2025
@joergmann joergmann closed this Aug 18, 2025
@joergmann joergmann deleted the test/changelogCheck branch August 18, 2025 07:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip changelog Do not require an entry in the changelog

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants