Skip to content

[pre-commit.ci] pre-commit autoupdate - #915

Open
pre-commit-ci[bot] wants to merge 3 commits into
mainfrom
pre-commit-ci-update-config
Open

pre-commit-ci[bot] wants to merge 3 commits into
mainfrom
pre-commit-ci-update-config

Conversation

updates:
- [github.com/astral-sh/ruff-pre-commit: v0.16.1 → v0.16.6](astral-sh/ruff-pre-commit@v0.16.1...v0.16.6)
- [github.com/shellcheck-py/shellcheck-py: v0.11.0.1 → v0.11.0.1-1](shellcheck-py/shellcheck-py@v0.11.0.1...v0.11.0.1-1)
- [github.com/zizmorcore/zizmor-pre-commit: v1.29.0 → v1.30.0](zizmorcore/zizmor-pre-commit@v1.29.0...v1.30.0)
@pre-commit-ci
pre-commit-ci Bot requested a review from a team as a code owner September 7, 2026 19:47
@pre-commit-ci
pre-commit-ci Bot requested a review from jameslamb September 7, 2026 19:47
@copy-pr-bot

copy-pr-bot Bot commented Sep 7, 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.

@jameslamb

This comment was marked as resolved.

@jameslamb

Copy link
Copy Markdown
Member

/ok to test 0135b37

fail-fast: false
secrets: inherit # zizmor: ignore[secrets-inherit]
uses: ./.github/workflows/build-rapids-image.yml
uses: $/.github/workflows/build-rapids-image.yml

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

The latest zizmor flags these.

help[self-repository]: use GitHub's dedicated self-repository syntax
  --> .github/workflows/publish.yml:30:11
   |
23 |   docker:
   |   ------ this job
...
30 |     uses: ./.github/workflows/build-test-publish-images.yml
   |           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ use '$/...' instead of './...'
   |
   = note: audit confidence → High
   = note: this finding has an auto-fix

I like it!

See https://github.blog/changelog/2026-07-30-reference-same-repository-actions-with-self-repository-syntax/

Part of the goal is to avoid source checkouts and unfortunately we still can't do that because we rely on parsing files in source control within these workflows. But still a nice change to adopt.

@jameslamb

Copy link
Copy Markdown
Member

/merge

@jameslamb jameslamb assigned jameslamb and unassigned jameslamb Sep 9, 2026
@jameslamb

Copy link
Copy Markdown
Member

This is failing like this:

22.62   - nothing provides libxgboost 3.3.0.* rapidsai_h* needed by rapids-xgboost-26.10.00a7-cuda12_260910192949_411cc434
22.62 
22.62 Could not solve for environment specs
22.62 The following package could not be installed
22.62 └─ rapids =26.10 * is not installable because it requires
22.62    └─ rapids-xgboost =26.10 *, which requires
22.62       └─ libxgboost =3.3.0 rapidsai_h*, which does not exist (perhaps a missing channel).

(build link)

Should be fixed once rapidsai/integration#868 is resolved.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant