Skip to content

[CI] Bump last occurrence of actions/checkout@v2 to v3#1303

Merged
sappelhoff merged 1 commit intobids-standard:masterfrom
DimitriPapadopoulos:actions
Sep 27, 2022
Merged

[CI] Bump last occurrence of actions/checkout@v2 to v3#1303
sappelhoff merged 1 commit intobids-standard:masterfrom
DimitriPapadopoulos:actions

Conversation

@DimitriPapadopoulos
Copy link
Copy Markdown
Collaborator

No description provided.

@codecov
Copy link
Copy Markdown

codecov bot commented Sep 26, 2022

Codecov Report

Base: 88.57% // Head: 88.57% // No change to project coverage 👍

Coverage data is based on head (56ce732) compared to base (a98ea74).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #1303   +/-   ##
=======================================
  Coverage   88.57%   88.57%           
=======================================
  Files           6        6           
  Lines        1042     1042           
=======================================
  Hits          923      923           
  Misses        119      119           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@Remi-Gau
Copy link
Copy Markdown
Collaborator

@DimitriPapadopoulos

I think our dependabot should take care of this.

- package-ecosystem: "github-actions"

Or at least should have. @sappelhoff Are we missing something?

Copy link
Copy Markdown
Member

@sappelhoff sappelhoff left a comment

Choose a reason for hiding this comment

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

yes, this should be taken care of by the bot 🤔 not sure why it wasn't triggered today, because it's set to "weekly" and it's Monday. 🤷‍♂️

I think we should rather fix the bot (potentially in this PR) than merge the manual changes

@DimitriPapadopoulos
Copy link
Copy Markdown
Collaborator Author

In past pull requests, I can see dependabot updating all actions except actions/checkout:

The merge request that introduced .github/dependabot.yml is #1168 from 1 Aug. At that point, all actions/checkout actions were already at @v3, after a manual update in #1044. The @v2 was introduced by #1252 on 25 Aug and never fixed by dependabot. Will have to find why...

@DimitriPapadopoulos
Copy link
Copy Markdown
Collaborator Author

I don't think I have access to dependabot logs to fix this. Someone else will have to look into this. Among possible reasons:

@sappelhoff
Copy link
Copy Markdown
Member

Where would I find the dependabot logs?

@sappelhoff sappelhoff added the exclude-from-changelog This item will not feature in the automatically generated changelog label Sep 27, 2022
@DimitriPapadopoulos
Copy link
Copy Markdown
Collaborator Author

DimitriPapadopoulos commented Sep 27, 2022

I don't have much experience with Dependabot. Perhaps Troubleshooting Dependabot errors will help:

Investigating errors with Dependabot security updates

When Dependabot is blocked from creating a pull request to fix a Dependabot alert, it posts the error message on the alert. The Dependabot alerts view shows a list of any alerts that have not been resolved yet. To access the alerts view, click Dependabot alerts on the Security tab for the repository.

Investigating errors with Dependabot version updates

When Dependabot is blocked from creating a pull request to update a dependency in an ecosystem, it posts the error icon on the manifest file. The manifest files that are managed by Dependabot are listed on the Dependabot tab. To access this tab, on the Insights tab for the repository click Dependency graph, and then click the Dependabot tab.

@DimitriPapadopoulos
Copy link
Copy Markdown
Collaborator Author

DimitriPapadopoulos commented Sep 27, 2022

Ah, based on the above, I think I do have access to the logs:
https://github.com/bids-standard/bids-specification/network/updates/469294046

Somehow, I think that because all other actions/checkout actions are already at version @v3, the single occurrence of @v2 goes unnoticed or is skipped on purpose:

INFO <job_469294046> Checking if actions/checkout 3 needs updating
  proxy | 2022/09/26 08:32:18 [015] GET https://github.com:443/actions/checkout.git/info/refs?service=git-upload-pack
  proxy | 2022/09/26 08:32:18 [015] * authenticating git server request (host: github.com)
  proxy | 2022/09/26 08:32:18 [015] 200 https://github.com:443/actions/checkout.git/info/refs?service=git-upload-pack
updater | INFO <job_469294046> Latest version is 3
updater | INFO <job_469294046> No update needed for actions/checkout 3

@sappelhoff
Copy link
Copy Markdown
Member

Curious, that seems like a bug in the dependabot script then! I am fine with merging this then, thanks a lot for looking into it @DimitriPapadopoulos

@sappelhoff sappelhoff merged commit 959b646 into bids-standard:master Sep 27, 2022
@DimitriPapadopoulos
Copy link
Copy Markdown
Collaborator Author

Yes, probably a bug. Strangely enough, I can see two sets of actions/checkout dependencies under tab Dependencies:
checkout

@DimitriPapadopoulos DimitriPapadopoulos deleted the actions branch September 27, 2022 09:51
@DimitriPapadopoulos
Copy link
Copy Markdown
Collaborator Author

After this request has been merged, the two distinct sets of actions/checkout dependencies have disappeared, but the count of actions/checkout dependencies still seems wrong to me (3 instead of 5):
checkout

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

exclude-from-changelog This item will not feature in the automatically generated changelog

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants