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

Don't use @master in your workflow #41

Closed
konradpabjan opened this issue Dec 17, 2019 · 4 comments
Closed

Don't use @master in your workflow #41

konradpabjan opened this issue Dec 17, 2019 · 4 comments

Comments

@konradpabjan
Copy link
Collaborator

konradpabjan commented Dec 17, 2019

During the beta of GitHub Actions, a lot of documentation (including the README for this action) showed example YAML with master being referenced:

uses: actions/upload-artifact@master

We currently encourage users to not use master as a reference whenever an action is being used. Instead, one of the available tags should be used such as:

uses: actions/upload-artifact@v1

Tags are used alongside semantic versioning to provide a stable experience: https://help.github.com/en/actions/automating-your-workflow-with-github-actions/about-actions#versioning-your-action

The master branch can abruptly change without notice during development which can cause workflows to unexpectedly fail. If you want to avoid unexpected changes or failures, you should use a tag when referring to a specific version of an action. Tags are added to stable versions that have undergone significant testing and should not change.

The v2 versions of download-artifact and upload-artifact are currently in development. Expect changes to start showing up in the v2-preview branch. These new changes will eventually be merged into master (we will communicate about this in the future) and that will have the potential to break your workflows if you are using @master so you will have to react by updating your YAML to use a tag.

Our telemetry indicates a significant amount of users are using @master. Good practice for all actions is to use a tag instead of referencing @master.

@konradpabjan konradpabjan pinned this issue Dec 17, 2019
@mittalyashu
Copy link

Can we subscribe to this issue for further updates for v2 release?

@konradpabjan
Copy link
Collaborator Author

v2 preview should (optimistic about this🤞) be coming out this week. I'll create another issue with updates and everything once that is out

@konradpabjan
Copy link
Collaborator Author

@mittalyashu The v2-preview is now out if you're interested 😀 (see the pinned issue in this repo). Took way way longer than I was expecting but it's out now.

Once v2-preview has been tested for some time, expect it to be merged into master

@mittalyashu
Copy link

Sorry for the trouble, thanks for the heads up 😁

@konradpabjan konradpabjan changed the title Using actions/upload-artifact@master in your workflow Don't use @master in your workflow Mar 19, 2020
dreamer added a commit to dosbox-staging/dosbox-staging that referenced this issue Jul 4, 2020
dreamer added a commit to dosbox-staging/dosbox-staging that referenced this issue Jul 5, 2020
New stable version of this action has ability to upload a single file
without explicitly naming it, but we can't use it because GitHub UI
creates a placeholder name.

Author of upload-artifact action clarified, that problem with
downloading "zip" files is not caused by action, but by GitHub UI,
therefore we can't remove unnecessary zip wrapper around Linux tarball
nor macOS dmg file still.

See: actions/upload-artifact#41
dreamer added a commit to dosbox-staging/dosbox-staging that referenced this issue Jul 5, 2020
New stable version of this action has ability to upload a single file
without explicitly naming it, but we can't use it because GitHub UI
creates a placeholder name.

Author of upload-artifact action clarified, that problem with
downloading "zip" files is not caused by action, but by GitHub UI,
therefore we can't remove unnecessary zip wrapper around Linux tarball
nor macOS dmg file still.

See: actions/upload-artifact#41
dreamer added a commit to dosbox-staging/dosbox-staging that referenced this issue Jul 5, 2020
New stable version of this action has ability to upload a single file
without explicitly naming it, but we can't use it because GitHub UI
creates a placeholder name.

Author of upload-artifact action clarified, that problem with
downloading "zip" files is not caused by action, but by GitHub UI,
therefore we can't remove unnecessary zip wrapper around Linux tarball
nor macOS dmg file still.

See: actions/upload-artifact#41
@konradpabjan konradpabjan unpinned this issue Jul 9, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants