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

Fix opam switch list-available when given several arguments #6318

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

kit-ty-kate
Copy link
Member

Required for #6186

Giving several arguments would never have given any results before. Furthermore it now matches the behaviour of opam list

@rjbou
Copy link
Collaborator

rjbou commented Dec 2, 2024

Agree on the idea. Some history, it was introduced from the beginning as an ands (see #2650).
We need to advertise it as it is a change of behaviour. Maybe have it behind a CLI version guard ?

master_changes.md Outdated Show resolved Hide resolved
@kit-ty-kate
Copy link
Member Author

Maybe have it behind a CLI version guard ?

i don't think we need to do that for something that had no useful outcome

@rjbou rjbou added the KIND: BUG label Dec 2, 2024
@rjbou
Copy link
Collaborator

rjbou commented Dec 2, 2024

As it is a bug fix, lgtm

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

Successfully merging this pull request may close these issues.

2 participants