Skip to content
New issue

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

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

Already on GitHub? # to your account

Lock pre-commit config to specific SHA #118

Merged
merged 1 commit into from
Jun 15, 2023
Merged

Conversation

jfrost-mo
Copy link
Member

@jfrost-mo jfrost-mo commented Jun 6, 2023

Security improvement suggested on Yammer. We where already pinning the versions, this just makes it more robust against malicious packages.

Fixes #117

@jfrost-mo jfrost-mo marked this pull request as ready for review June 6, 2023 11:43
@github-actions
Copy link
Contributor

github-actions bot commented Jun 6, 2023

Coverage

@jfrost-mo jfrost-mo added the cleanup Non-functional improvement label Jun 6, 2023
Copy link
Contributor

@Sylviabohnenstengel Sylviabohnenstengel left a comment

Choose a reason for hiding this comment

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

happy with change

@Sylviabohnenstengel Sylviabohnenstengel merged commit 309fac6 into main Jun 15, 2023
@Sylviabohnenstengel Sylviabohnenstengel deleted the jfrost-mo-patch-1 branch June 15, 2023 09:49
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
cleanup Non-functional improvement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Freeze pre-commit config
2 participants