Skip to content

build: use path-ignore in GHA coverage-windows.yml#61811

Open
legendecas wants to merge 1 commit intonodejs:mainfrom
legendecas:gha-win
Open

build: use path-ignore in GHA coverage-windows.yml#61811
legendecas wants to merge 1 commit intonodejs:mainfrom
legendecas:gha-win

Conversation

@legendecas
Copy link
Member

@legendecas legendecas commented Feb 13, 2026

Use similar paths-ignore in other GHA jobs like .github/workflows/test-macos.yml to ignore certain changes that do not have effect on builds.

This allows changes to configure.py, node.gyp, or src/inspector/node_protocol.pdl, etc, to be tested on GHA windows.

For example, windows GHA was not triggered on #61806.

@nodejs-github-bot
Copy link
Collaborator

Review requested:

  • @nodejs/actions

@nodejs-github-bot nodejs-github-bot added the meta Issues and PRs related to the general management of the project. label Feb 13, 2026
@legendecas legendecas changed the title build: run GHA windows test when gyp file changes build: use path-ignore in GHA coverage-windows.yml Feb 13, 2026
@aduh95
Copy link
Contributor

aduh95 commented Feb 13, 2026

This allows changes to configure.py, node.gyp, or src/inspector/node_protocol.pdl, etc, to be tested on GHA windows.

A test failure doesn't mark the job as failure, no? In this case, is it worth it? I don't think changes on those files would affect the coverage, but could be wrong

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

meta Issues and PRs related to the general management of the project.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants