Skip to content

chore(pre-commit): add actionlint pre-commit-config - #1018

Merged
kkraus14 merged 3 commits into
NVIDIA:mainfrom
cpcloud:actionlint
Sep 26, 2025
Merged

chore(pre-commit): add actionlint pre-commit-config#1018
kkraus14 merged 3 commits into
NVIDIA:mainfrom
cpcloud:actionlint

Conversation

@cpcloud

@cpcloud cpcloud commented Sep 25, 2025

Copy link
Copy Markdown
Contributor

Add actionlint pre-commit-config step.

@copy-pr-bot

copy-pr-bot Bot commented Sep 25, 2025

Copy link
Copy Markdown
Contributor

Auto-sync is disabled for ready for review pull requests in this repository. Workflows must be run manually.

Contributors can view more details about this message here.

@cpcloud

cpcloud commented Sep 25, 2025

Copy link
Copy Markdown
Contributor Author

This would've caught my misspelling:

image

name: "CI: Release"

description: Manually-triggered release workflow. Creates a release draft if one doesn't exist for the given tag, or uses existing draft.
# Manually-triggered release workflow. Creates a release draft if one doesn't exist for the given tag, or uses existing draft.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

description is only a valid field in action.ymls, not workflows.

Comment thread .pre-commit-config.yaml Outdated
@cpcloud

cpcloud commented Sep 25, 2025

Copy link
Copy Markdown
Contributor Author

@kkraus14 Are you comfortable merging as-is, or should we do a test run?

@kkraus14

Copy link
Copy Markdown
Collaborator

/ok to test

@kkraus14
kkraus14 enabled auto-merge (squash) September 26, 2025 00:57
@github-actions

This comment has been minimized.

@kkraus14
kkraus14 merged commit f9b4930 into NVIDIA:main Sep 26, 2025
70 checks passed
@github-actions

Copy link
Copy Markdown
Doc Preview CI
Preview removed because the pull request was closed or merged.

@cpcloud
cpcloud deleted the actionlint branch September 26, 2025 16:12
@leofang leofang added enhancement Any code-related improvements CI/CD CI/CD infrastructure labels Oct 9, 2025
@leofang leofang added this to the cuda.core beta 7 milestone Oct 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI/CD CI/CD infrastructure enhancement Any code-related improvements

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants