Skip to content

feat(github-actions): Support for short tag comments #18738

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

Merged
merged 9 commits into from
Nov 4, 2022

Conversation

zharinov
Copy link
Collaborator

@zharinov zharinov commented Nov 3, 2022

Changes

Context

Documentation (please check one with an [x])

  • I have updated the documentation, or
  • No documentation update is required

How I've tested my work (please tick one)

I have verified these changes via:

  • Code inspection only, or
  • Newly added/modified unit tests, or
  • No unit tests but ran on a real repository, or
  • Both unit tests + ran on a real repository

@zharinov
Copy link
Collaborator Author

zharinov commented Nov 3, 2022

Copy link
Member

@viceice viceice left a comment

Choose a reason for hiding this comment

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

Otherwise LGTM

@zharinov zharinov requested a review from rarkins November 3, 2022 15:08
Copy link
Collaborator

@rarkins rarkins left a comment

Choose a reason for hiding this comment

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

What affect does this have on when we "pin" actions to a digest? Do we still use tag= in that case?

@zharinov
Copy link
Collaborator Author

zharinov commented Nov 4, 2022

I'm not sure what do you mean, for me this PR just extends the syntax we're able to parse.

@rarkins
Copy link
Collaborator

rarkins commented Nov 4, 2022

We have functionality to convert from tag-only to sha + tag comment, when pinDigests=true

@rarkins rarkins requested a review from viceice November 4, 2022 09:06
Copy link
Member

@viceice viceice left a comment

Choose a reason for hiding this comment

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

otherwise LGTM

@zharinov zharinov requested review from rarkins and viceice November 4, 2022 10:25
@viceice viceice merged commit 290b964 into renovatebot:main Nov 4, 2022
@viceice viceice deleted the feat/github-actions-short-tags branch November 4, 2022 12:20
@renovate-release
Copy link
Collaborator

🎉 This PR is included in version 34.18.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@Gabriel-Ladzaretti
Copy link
Collaborator

Gabriel-Ladzaretti commented Nov 15, 2022

@viceice @zharinov
i ran this againt the full dependabot yaml file, is the short sha to semver update the expected behavior?

image

EDIT: also note that line 16 (short sha) is expended into full sha

@Gabriel-Ladzaretti
Copy link
Collaborator

Gabriel-Ladzaretti commented Nov 15, 2022

ok so i ran this with an older version of renovate
image

so this is not related to this one, but still is this expected 🙃:?

looking at line 16, it looks like it does 🤔 (?)

@zharinov
Copy link
Collaborator Author

Tell me what's expected, I'll fix

@viceice
Copy link
Member

viceice commented Nov 15, 2022

  • line 16 is wrong, it should update to long sha and not to short version. best to even not touch at all if sha is same
  • line 19 seems to be wrong. the comment says it shouldn't be updated by renovate. 🤔

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Dec 16, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Support "short" GitHub Actions tag comments
5 participants