Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Add dependencies only switch import #5388

Merged
merged 3 commits into from
Jul 24, 2024
Merged

Add dependencies only switch import #5388

merged 3 commits into from
Jul 24, 2024

Conversation

rjbou
Copy link
Collaborator

@rjbou rjbou commented Dec 14, 2022 •
edited
Loading

Flag --deps-only for opam import that install only non root & compiler packages.
Fixes #5200

@rjbou rjbou added the PR: QUEUED Pending pull request, waiting for other work to be merged or closed label Dec 14, 2022
@rjbou rjbou changed the title Add dependence only import of a switch Add dependencies only switch import Dec 14, 2022
@rjbou rjbou marked this pull request as draft July 10, 2024 14:17
@rjbou rjbou requested review from kit-ty-kate and dra27 July 24, 2024 17:39
@rjbou rjbou added PR: WAITING FOR REVIEW and removed PR: QUEUED Pending pull request, waiting for other work to be merged or closed labels Jul 24, 2024
@rjbou rjbou marked this pull request as ready for review July 24, 2024 17:39
kit-ty-kate
kit-ty-kate approved these changes Jul 24, 2024
View reviewed changes
Copy link
Member

@kit-ty-kate kit-ty-kate left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

i've split and improved the test in a separate commit. lgtm

@kit-ty-kate
Copy link
Member

Thanks!

@kit-ty-kate kit-ty-kate merged commit 1c85401 into ocaml:master Jul 24, 2024
29 checks passed
@kit-ty-kate kit-ty-kate added this to the 2.3.0~alpha milestone Jul 24, 2024
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@kit-ty-kate kit-ty-kate kit-ty-kate approved these changes

@dra27 dra27 Awaiting requested review from dra27

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
2.3.0~alpha1
Development

Successfully merging this pull request may close these issues.

feature request: opam switch import --deps-only
2 participants
@rjbou @kit-ty-kate