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

Github Action: Doesn't fail on invalid lockfile #880

Open
LogFlames opened this issue Sep 8, 2024 · 3 comments
Open

Github Action: Doesn't fail on invalid lockfile #880

LogFlames opened this issue Sep 8, 2024 · 3 comments

Comments

@LogFlames
Copy link
Member

From the test #879 the github action don't seem to fail when "commit-lockfile" is set to false and the lockfile is invalid

@LogFlames
Copy link
Member Author

This was a misunderstanding from my side. It tries to push the updated lockfile (only if it is changed), and if it failed to push it causes the action to fail.

The parameter "commit-lockfile" must therefore be set to true, for the process to fail in case of lockfile error

@LogFlames LogFlames reopened this Sep 8, 2024
@LogFlames
Copy link
Member Author

LogFlames commented Sep 8, 2024

I think this is unexpected behaviour.

If "commit-lockfile" is false, my expectation is that it verifies the lockfile and succeed if it is correct and fail if it is incorrect, without trying to push the new lockfile to the branch.

Would love a second perspective!

@monperrus
Copy link
Contributor

where is the reference documentation of flag "commit-lockfile"? If it does not exist, we have to write it first and settle on the reference behavior.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants