Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Various hardenings#170

Merged
woodruffw merged 1 commit into
mainfrom
workflow-cleanup
Jun 10, 2026
Merged

Various hardenings#170
woodruffw merged 1 commit into
mainfrom
workflow-cleanup

Conversation

@AlexWaygood

Copy link
Copy Markdown
Member

Summary

  • Fix actionlint warnings (from its shellcheck integration) due to various variables not being quoted in main.yml
  • Remove the repository_dispatch trigger in main.yml. The Ruff workflow that triggers new releases in this repository has used workflow_dispatch since 2023: Refactor and fix task trigger for dependent jobs in other repos ruff#4598.
  • Add a dependabot cooldown to fix zizmor warnings.

I will apply these changes to our other pre-commit repos too, if this is accepted.

@AlexWaygood AlexWaygood requested a review from woodruffw June 10, 2026 14:48
@AlexWaygood

Copy link
Copy Markdown
Member Author

(I think it would probably be good to run these linters in CI, but exactly how we do that might be controversial, so I'll do that separately)

@woodruffw woodruffw left a comment

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.

Nice!

@woodruffw

Copy link
Copy Markdown
Member

(I think it would probably be good to run these linters in CI, but exactly how we do that might be controversial, so I'll do that separately)

Fully agreed on running them in CI 🙂

@woodruffw woodruffw merged commit 99e6029 into main Jun 10, 2026
@woodruffw woodruffw deleted the workflow-cleanup branch June 10, 2026 17:32
AlexWaygood added a commit to astral-sh/ty-pre-commit that referenced this pull request Jun 10, 2026
AlexWaygood added a commit to astral-sh/uv-pre-commit that referenced this pull request Jun 10, 2026
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