Skip to content

ci: skip branch commit guard in lint workflow#77

Merged
qartik merged 1 commit into
mainfrom
ks-skip-no-commit-branch-ci
Apr 29, 2026
Merged

ci: skip branch commit guard in lint workflow#77
qartik merged 1 commit into
mainfrom
ks-skip-no-commit-branch-ci

Conversation

@qartik
Copy link
Copy Markdown
Member

@qartik qartik commented Apr 29, 2026

Summary

  • Skip the no-commit-to-branch hook when CI runs the full prek lint suite.
  • Keep the hook active for local commits while avoiding failures on regular main push runs.

Fixes the main push CI failure from run 25113921477.

Validation

  • SKIP=no-commit-to-branch uvx prek run --all-files
  • make test

@qartik qartik marked this pull request as ready for review April 29, 2026 15:35
Copilot AI review requested due to automatic review settings April 29, 2026 15:35
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the CI lint workflow to avoid failing on main pushes due to the no-commit-to-branch pre-commit hook, while keeping that hook active for local developer workflows.

Changes:

  • Sets SKIP=no-commit-to-branch for the CI “Run pre-commit lint” step.
  • Keeps the rest of the uvx prek run --all-files lint suite unchanged.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@qartik qartik merged commit 6774243 into main Apr 29, 2026
14 checks passed
@qartik qartik deleted the ks-skip-no-commit-branch-ci branch April 29, 2026 16:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants