-
Notifications
You must be signed in to change notification settings - Fork 5
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
AN-301 Update certifi #795
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for doing this, looks like some python setup fun. I have one minor request for adding a comment but should not be a blocker for merging.
@@ -19,7 +19,7 @@ run_secret_setup: &run_secret_setup | |||
command: | | |||
git clone https://github.com/awslabs/git-secrets.git | |||
pushd git-secrets >/dev/null | |||
git checkout ad82d68ee924906a0401dfd48de5057731a9bc84 | |||
git checkout 5357e18bc27b42a827b6780564ea873a72ca1f01 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could you add a comment explaining what pointing to this specific commit does?
.circleci/config.yml
Outdated
@@ -28,7 +28,7 @@ jobs: | |||
<<: *job_defaults | |||
machine: | |||
# latest pinned 20.04 version https://circleci.com/developer/machine/image/ubuntu-2004 | |||
image: ubuntu-2004:202201-02 | |||
image: ubuntu-2004:2024.11.1 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could be worth updating to the latest LTS.
20.04 maintenance ends in April 2025.
image: ubuntu-2004:2024.11.1 | |
image: ubuntu-2404:2024.11.1 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good call!
No description provided.