Skip to content

fix: ensure pre-commit is using latest version via HTTPS #240

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 3 commits into from
Nov 22, 2021
Merged

fix: ensure pre-commit is using latest version via HTTPS #240

merged 3 commits into from
Nov 22, 2021

Conversation

@bryantbiggs
Copy link
Member

@bryantbiggs bryantbiggs commented Nov 22, 2021

Description

  • ensure pre-commit is using https for pulling hooks

Motivation and Context

  • standardize with rest of repositories

Breaking Changes

  • No

How Has This Been Tested?

  • N/A

@antonbabenko antonbabenko merged commit 57935f0 into terraform-aws-modules:master Nov 22, 2021
@bryantbiggs bryantbiggs deleted the fix/update-ci branch November 22, 2021 16:15
antonbabenko pushed a commit that referenced this pull request Dec 6, 2021
@semantic-release-bot
chore(release): version 3.5.3 [skip ci]
167af83 
## [3.5.3](v3.5.2...v3.5.3) (2021-12-06)

### Bug Fixes

* atlantis health check path ([#242](#242)) ([0b17a41](0b17a41))
* ensure pre-commit is using latest version via HTTPS ([#240](#240)) ([57935f0](57935f0))
@antonbabenko
Copy link
Member

antonbabenko commented Dec 6, 2021

This PR is included in version 3.5.3 🎉

@github-actions
Copy link

github-actions bot commented Nov 9, 2022

I'm going to lock this pull request because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues. If you have found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Nov 9, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@bryantbiggs @antonbabenko