-
Notifications
You must be signed in to change notification settings - Fork 605
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
Use fork markers and fork preferences in resolution with lockfile #5481
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
konstin
force-pushed
the
konsti/add-forks-to-lockfile
branch
from
July 26, 2024 14:33
d5c695d
to
2059be1
Compare
konstin
force-pushed
the
konsti/merge-identical-resolutions
branch
3 times, most recently
from
July 26, 2024 14:54
3995731
to
157f10b
Compare
konstin
force-pushed
the
konsti/add-forks-to-lockfile
branch
from
July 26, 2024 15:53
2059be1
to
f149824
Compare
konstin
force-pushed
the
konsti/merge-identical-resolutions
branch
from
July 26, 2024 15:53
157f10b
to
ff4651c
Compare
konstin
force-pushed
the
konsti/add-forks-to-lockfile
branch
from
July 29, 2024 12:02
f149824
to
9ff639c
Compare
konstin
force-pushed
the
konsti/merge-identical-resolutions
branch
from
July 29, 2024 12:02
ff4651c
to
6f29c03
Compare
konstin
force-pushed
the
konsti/add-forks-to-lockfile
branch
from
July 29, 2024 12:14
9ff639c
to
759a1be
Compare
konstin
force-pushed
the
konsti/merge-identical-resolutions
branch
from
July 29, 2024 12:18
6f29c03
to
171dcb4
Compare
konstin
force-pushed
the
konsti/add-forks-to-lockfile
branch
from
July 29, 2024 13:10
759a1be
to
287463f
Compare
konstin
force-pushed
the
konsti/merge-identical-resolutions
branch
from
July 29, 2024 13:13
171dcb4
to
a1e82fd
Compare
konstin
force-pushed
the
konsti/add-forks-to-lockfile
branch
3 times, most recently
from
July 29, 2024 13:50
8aff9ae
to
f521fdf
Compare
konstin
force-pushed
the
konsti/merge-identical-resolutions
branch
from
July 29, 2024 13:50
a1e82fd
to
662f87d
Compare
konstin
force-pushed
the
konsti/add-forks-to-lockfile
branch
from
July 30, 2024 10:58
f521fdf
to
4b1ca1f
Compare
konstin
force-pushed
the
konsti/merge-identical-resolutions
branch
from
July 31, 2024 10:53
662f87d
to
100e160
Compare
BurntSushi
approved these changes
Jul 31, 2024
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I buy it!
With this change, the "preferences-dependent-forking" packse scenario becomes stable.
konstin
force-pushed
the
konsti/merge-identical-resolutions
branch
from
July 31, 2024 15:09
b0e3552
to
87ead41
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
By resolving for each fork from the lockfile individually and by adding using preferences for the current fork, we solve the instability #5180. I've tested the locally and will add the packse test scenarios upstack.
Part of #5180 (comment)