Cargo.lock in repository #368
Closed
jaybosamiya
started this conversation in
General
Replies: 1 comment
-
|
Being added by @jaybosamiya in #669. |
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
-
At some point, we should consider maintaining a
Cargo.lockin the repository.It was originally
.gitignored in 2021: d4ded8f#360 tried to (re)introduce it (including some rationale for why we should be including
Cargo.lock). Since non-trivial changes were soon to be made (to move off the rustc fork), addingCargo.lockright now as decided against.Likely, once we have moved off of the rustc fork, we should revisit this and lock our dependencies to make longer term maintenance easier.
If any more reasons (other than the rustc fork move) exist to not keep the
Cargo.lockaround, it would be good to hear them, thus this discussion thread. For reasons to maintainCargo.lock, some are already listed on #360.Beta Was this translation helpful? Give feedback.
All reactions