Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ci(lint): pin differential-shellcheck to v3 tag #19

Merged
merged 1 commit into from
May 25, 2023
Merged

ci(lint): pin differential-shellcheck to v3 tag #19

merged 1 commit into from
May 25, 2023

Conversation

jamacku
Copy link
Contributor

@jamacku jamacku commented Dec 9, 2022

Please consider using the v3 tag instead of latest. We expect that a new major release could break the API of the differential-shellcheck GitHub Action. I have also removed write permissions since they are required only for private repositories.

Related to:

@5long 5long merged commit f43501e into 5long:master May 25, 2023
@jamacku jamacku deleted the patch-1 branch May 25, 2023 18:44
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