Skip to content

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

Merged
jameslamb merged 2 commits into
mainfrom
pre-commit-ci-update-config
Oct 7, 2026
Merged

jameslamb merged 2 commits into
mainfrom
pre-commit-ci-update-config

Conversation

@pre-commit-ci

@pre-commit-ci pre-commit-ci Bot commented Sep 7, 2026 •

Copy link
Copy Markdown
Contributor

@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.

updates:
- [github.com/astral-sh/ruff-pre-commit: v0.16.1 → v0.16.10](astral-sh/ruff-pre-commit@v0.16.1...v0.16.10)
- [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.1](zizmorcore/zizmor-pre-commit@v1.29.0...v1.30.1)
- [github.com/rapidsai/pre-commit-hooks: v1.6.1 → v1.7.0](rapidsai/pre-commit-hooks@v1.6.1...v1.7.0)
@pre-commit-ci
pre-commit-ci Bot force-pushed the pre-commit-ci-update-config branch from 0135b37 to f91002d Compare October 5, 2026 21:03
@jameslamb

Copy link
Copy Markdown
Member

/ok to test d51b51b

@jameslamb

Copy link
Copy Markdown
Member

The check-nightly-ci failures here are weird.

The job was not started because it repeatedly failed to be acquired (5 attempts).
The job was not started because it repeatedly failed to be acquired (5 attempts).
The job was not started because it repeatedly failed to be acquired (5 attempts).
The job was not started because it repeatedly failed to be acquired (5 attempts).
The job was not started because it repeatedly failed to be acquired (5 attempts).
The job was not started because it repeatedly failed to be acquired (5 attempts).
Internal server error. Correlation ID: caae535a-d1dc-401a-ad76-b06395c95916

https://github.com/rapidsai/docker/actions/runs/37655492666

Might be an artifact of the ongoing GitHub outages the last few days.

This PR only touches pre-commit configuration and pre-commit.ci passed, so I'm going to admin merge this. If we see more PR CI failing we can address that separately.

@jameslamb
jameslamb merged commit 87b8bb2 into main Oct 7, 2026
4 of 5 checks passed
@jameslamb
jameslamb deleted the pre-commit-ci-update-config branch October 7, 2026 20:10
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