Skip to content

Conversation

@markg-github
Copy link
Contributor

No description provided.

Copy link
Collaborator

@npmccallum npmccallum left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please also squash your commits.

@markg-github markg-github force-pushed the improve/gh_authentication branch from 94572ac to b84ee7c Compare October 17, 2025 14:14
@markg-github markg-github force-pushed the improve/gh_authentication branch from b84ee7c to 7a51aa5 Compare October 17, 2025 15:08
Copy link
Collaborator

@npmccallum npmccallum left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

improve is not a valid conventional commit type. See https://www.conventionalcommits.org/en/v1.0.0/#summary

@npmccallum npmccallum force-pushed the improve/gh_authentication branch from 7a51aa5 to 75d980f Compare October 17, 2025 15:54
@npmccallum
Copy link
Collaborator

@markg-github You can see I've added a linter for conventional commit syntax and your PR now doesn't pass that test.

Improve dispatch error output when dispatch fails to authenticate with
GitHub by describing permissions needed to authenticate.
@markg-github markg-github force-pushed the improve/gh_authentication branch from 75d980f to 4e199d0 Compare October 17, 2025 16:32
@markg-github
Copy link
Contributor Author

@npmccallum should be good now.

@npmccallum npmccallum enabled auto-merge (rebase) October 17, 2025 17:01
@npmccallum npmccallum merged commit 0b5c423 into AMDEPYC:main Oct 17, 2025
8 checks passed
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