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: use concrete action version #172

Merged
merged 1 commit into from
Sep 11, 2023

Conversation

lgoettgens
Copy link
Collaborator

This is recommended by the readme.

@codecov
Copy link

codecov bot commented Sep 5, 2023

Codecov Report

Merging #172 (d04201f) into master (8dc359d) will increase coverage by 0.03%.
Report is 2 commits behind head on master.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #172      +/-   ##
==========================================
+ Coverage   77.58%   77.61%   +0.03%     
==========================================
  Files          11       11              
  Lines         736      746      +10     
==========================================
+ Hits          571      579       +8     
- Misses        165      167       +2     
Flag Coverage Δ
unittests 77.61% <ø> (+0.03%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

see 2 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@fingolfin fingolfin merged commit 3cd8d0e into JuliaTesting:master Sep 11, 2023
18 checks passed
@lgoettgens lgoettgens deleted the lg/action-v branch September 11, 2023 09:52
lgoettgens added a commit that referenced this pull request Sep 19, 2023
@3f6a
Copy link

3f6a commented Sep 19, 2023

Actually the recommendation is to use SHA of the commit:

If your workflow requires access to secrets, you should always pin it to a commit SHA instead of a tag. This will protect you in case a bad actor gains access to the setup-julia repo. You can find more information in GitHub's security hardening guide.

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.

3 participants