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

Unable to resolve action ScottBrenner/cfn-lint-action@master, unable to find version master #37

Closed
NickLiffen opened this issue Apr 19, 2021 · 7 comments
Assignees
Labels
bug Something isn't working

Comments

@NickLiffen
Copy link
Collaborator

NickLiffen commented Apr 19, 2021

Getting the following error:

Unable to resolve action `ScottBrenner/cfn-lint-action@master`, unable to find version `master`

@ScottBrenner we may want to revert back to master, or create a separate branch called master and then when/if we release #33 why don't we bring in the removal of the master branch with this breaking change? (If that makes sense? Thoughts?

@ScottBrenner ScottBrenner self-assigned this Apr 19, 2021
@ScottBrenner ScottBrenner added the bug Something isn't working label Apr 19, 2021
@ScottBrenner
Copy link
Owner

Oops, reverted the branch name 😰

@hoxu
Copy link

hoxu commented Apr 20, 2021

@ScottBrenner looks like the master branch is still missing.

Renaming the branch for a repository that is referenced from god knows how many places is a very bad idea.

@ScottBrenner
Copy link
Owner

Normally I would agree, but 15421eb introduced breaking changes - regardless of branch name, the Action would and did break for anyone not version pinning to a specific release.

@hoxu
Copy link

hoxu commented Apr 20, 2021

One option would be to leave master pointing to the last commit before breaking changes. That would allow people to upgrade to the recommended v2 tagging on their leisure. GitHub will show the default branch on next visit, at which point they would discover about v2.

@hoxu
Copy link

hoxu commented Apr 20, 2021

Another consideration: organizations that use whitelisted actions can't make this upgrade without inertia:

scottbrenner/cfn-lint-action@v2 is not allowed to be used in XXX. Actions in this workflow must be: within a repository that belongs to your Enterprise account, created by GitHub, verified in the GitHub Marketplace or match the following XXX

@ScottBrenner
Copy link
Owner

Fair points, I've just revived the "old" Action in https://github.com/ScottBrenner/cfn-lint-action/tree/master - but know that it's deprecated! 😉

@hoxu
Copy link

hoxu commented Apr 20, 2021

Thanks! This will allow us to run the workflow until we get v2 whitelisted :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants