-
Notifications
You must be signed in to change notification settings - Fork 53
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
Lock file is updated during install with multiple tags for the same commit #403
Comments
Thanks for opening this issue
Is this what you are experiencing? |
@davidreuss Can you please confirm if it is the above behavior you are experiencing, |
Sorry, this got lost in my notifications. Yes, that is correct, and what i observed. 👍 There's a lot of scenarios that could trigger similar behaviour, but the important take away is that i do not expect the lock file to change, when i'm only installing dependencies (using In short, i don't expect the lock file to change when consulting it for installing dependencies. |
What steps did you take:
I have a repository, where we keep a "floating" major version say
v1
for my project (for referencing with github actions for instance), but it's also tagged with the exact semver version.What happened:
When running
vendir sync --locked
i'm seeing thevendir.lock.yml
file updated to reflect a difference in thetags
property of a dependency.We have CI jobs which checks that there are no differences in lock files, to determine if there's something changed or not, and this check is now failing randomly depending on when the last vendir sync was executed, and what the state of the tags in the upstream project is at the time.
An example diff of what i observed below:
What did you expect:
I expect the lockfile to not be updated when i'm installing dependencies from that lock file, with
vendir sync --locked
Environment:
This is on mac os but i don't think there's any platform/arch issue going on here.
Vote on this request
This is an invitation to the community to vote on issues, to help us prioritize our backlog. Use the "smiley face" up to the right of this comment to vote.
👍 "I would like to see this addressed as soon as possible"
👎 "There are other more important things to focus on right now"
We are also happy to receive and review Pull Requests if you want to help working on this issue.
The text was updated successfully, but these errors were encountered: