Skip to content

Use major version ref of carlosperate/download-file-action #44

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 1 commit into from
Dec 20, 2021
Merged

Use major version ref of carlosperate/download-file-action #44

merged 1 commit into from
Dec 20, 2021

Conversation

per1234
Copy link
Contributor

@per1234 per1234 commented Dec 20, 2021

The carlosperate/download-file-action action is used by the "Sync Labels" GitHub Actions workflow as a convenient way
to download external resources.

A major version ref has been added to that repository:
https://github.com/carlosperate/download-file-action/releases/tag/v1
This mean it is no longer necessary to do a full pin of the action version in use as before.

Use of the major version ref will cause the workflow to use a stable version of the action, while also benefiting from ongoing development to the action up until such time as a new major release of an action is made, at which time we would need to evaluate whether any changes to the workflow are required by the breaking change that triggered the major release before updating the major ref (e.g., uses: carlosperate/download-file-action@v2). I think this strikes the right balance between stability and maintainability for the workflow.


Supersedes #43

The `carlosperate/download-file-action` action is used by the "Sync Labels" GitHub Actions workflow as a convenient way
to download external resources.

A major version ref has been added to that repository, meaning it is no longer necessary to do a full pin of the action
version in use as before.

Use of the major version ref will cause the workflow to use a stable version of the action, while also benefiting from
ongoing development to the action up until such time as a new major release of an action is made, at which time we would
need to evaluate whether any changes to the workflow are required by the breaking change that triggered the major
release before updating the major ref (e.g., uses: `carlosperate/download-file-action@v2`). I think this strikes the
right balance between stability and maintainability for the workflow.
@per1234 per1234 added type: enhancement Proposed improvement topic: infrastructure Related to project infrastructure labels Dec 20, 2021
@per1234 per1234 requested a review from aentinger December 20, 2021 03:11
Copy link
Contributor

@aentinger aentinger left a comment

Choose a reason for hiding this comment

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

LGTM 👍 Thank you @per1234 🚀 , this is good to know. I'll take it into consideration when adding sync-labels to new repositories 👍

@aentinger aentinger merged commit 290d01e into arduino-libraries:master Dec 20, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
topic: infrastructure Related to project infrastructure type: enhancement Proposed improvement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants