Skip to content

Pin 3rd party CI action version#10481

Merged
Fokko merged 1 commit into
apache:mainfrom
findepi:findepi/pin-3rd-party-ci-action-version-179c72
Jun 11, 2024
Merged

Pin 3rd party CI action version#10481
Fokko merged 1 commit into
apache:mainfrom
findepi:findepi/pin-3rd-party-ci-action-version-179c72

Conversation

@findepi
Copy link
Copy Markdown
Member

@findepi findepi commented Jun 11, 2024

GitHub allows to delete and re-publish a tag, so referencing 3rd party action by tag name should be discouraged.

GitHub allows to delete and re-publish a tag, so referencing 3rd party
action by tag name should be discouraged.
@github-actions github-actions Bot added the INFRA label Jun 11, 2024
Copy link
Copy Markdown
Contributor

@Fokko Fokko left a comment

Choose a reason for hiding this comment

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

LGTM, thanks @findepi for fixing this

The tag and hash: https://github.com/jlumbroso/free-disk-space/releases/tag/v1.3.1

@Fokko Fokko merged commit 282fa73 into apache:main Jun 11, 2024
@findepi findepi deleted the findepi/pin-3rd-party-ci-action-version-179c72 branch June 11, 2024 12:39
findepi added a commit to findepi/iceberg that referenced this pull request Jun 17, 2024
jasonf20 pushed a commit to jasonf20/iceberg that referenced this pull request Aug 4, 2024
GitHub allows to delete and re-publish a tag, so referencing 3rd party
action by tag name should be discouraged.
sasankpagolu pushed a commit to sasankpagolu/iceberg that referenced this pull request Oct 27, 2024
GitHub allows to delete and re-publish a tag, so referencing 3rd party
action by tag name should be discouraged.
zachdisc pushed a commit to zachdisc/iceberg that referenced this pull request Dec 23, 2024
GitHub allows to delete and re-publish a tag, so referencing 3rd party
action by tag name should be discouraged.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants