Skip to content

Fix docs-build changed-files dependency - #2636

Open
vuule wants to merge 3 commits into
NVIDIA:mainfrom
vuule:fix-docs-build-changed-files-dependency
Open

vuule wants to merge 3 commits into
NVIDIA:mainfrom
vuule:fix-docs-build-changed-files-dependency

Conversation

@vuule

@vuule vuule commented Sep 16, 2026

Copy link
Copy Markdown

Declares changed-files as a docs-build dependency so the build_docs output is available when GitHub evaluates the job condition. Without this dependency, documentation builds are skipped even when build_docs is true.

Signed-off-by: Vukasin Milovanovic <vmilovanovic@nvidia.com>
@copy-pr-bot

copy-pr-bot Bot commented Sep 16, 2026

Copy link
Copy Markdown

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

@bdice
bdice marked this pull request as ready for review September 16, 2026 21:51
@bdice
bdice requested a review from a team as a code owner September 16, 2026 21:51
@bdice
bdice requested a review from gforsyth September 16, 2026 21:51
@bdice bdice added improvement Improves an existing functionality non-breaking Introduces a non-breaking change labels Sep 16, 2026
@bdice

bdice commented Sep 16, 2026

Copy link
Copy Markdown
Contributor

/merge

@bdice

bdice commented Sep 16, 2026

Copy link
Copy Markdown
Contributor

/ok to test

@copy-pr-bot

copy-pr-bot Bot commented Sep 16, 2026

Copy link
Copy Markdown

/ok to test

@bdice, there was an error processing your request: E1

See the following link for more information: https://docs.gha-runners.nvidia.com/cpr/e/1/

@bdice

bdice commented Sep 16, 2026

Copy link
Copy Markdown
Contributor

/ok to test b115b88

@bdice

bdice commented Sep 17, 2026

Copy link
Copy Markdown
Contributor

/merge

@vuule

vuule commented Sep 17, 2026

Copy link
Copy Markdown
Author

@bdice I think that another test trigger comment is needed here

@bdice

bdice commented Sep 17, 2026

Copy link
Copy Markdown
Contributor

/ok to test 5ff60e6

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

Labels

improvement Improves an existing functionality non-breaking Introduces a non-breaking change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants