Skip to content

Update CONTRIBUTING.md#3794

Merged
davecramer merged 2 commits intomasterfrom
release-yml
Sep 10, 2025
Merged

Update CONTRIBUTING.md#3794
davecramer merged 2 commits intomasterfrom
release-yml

Conversation

@davecramer
Copy link
Member

fix the name of release.yml

fix the name of release.yml
@davecramer
Copy link
Member Author

@vlsi why are the pending checks not running?

@sehrope
Copy link
Member

sehrope commented Sep 9, 2025

I think it's not running because of the path filter excluding docs-only changes:

paths-ignore:
- 'docs/**'
- '**/*.md'
- 'appveyor.xml'

But I guess it's marked as required at the repo level. So it doesn't consider the PR ready to go, but it will never actually try to run the (pointless) style check.

@davecramer
Copy link
Member Author

I think it's not running because of the path filter excluding docs-only changes:

paths-ignore:
- 'docs/**'
- '**/*.md'
- 'appveyor.xml'

But I guess it's marked as required at the repo level. So it doesn't consider the PR ready to go, but it will never actually try to run the (pointless) style check.

I figured as much.

@davecramer davecramer merged commit 549576c into master Sep 10, 2025
6 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

Comments