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

Rename opam list --silent to opam list --check #4595

Merged
merged 4 commits into from
Mar 23, 2021
Merged

Conversation

dra27
Copy link
Member

@dra27 dra27 commented Mar 11, 2021

This is sat on #4581 and #4575 both of which need merging first. The actual commit itself is tiny - --silent is of course retained in the default CLI.

Fixes #4323.

@dra27 dra27 added this to the 2.1.0~rc milestone Mar 11, 2021
src/client/opamCommands.ml Outdated Show resolved Hide resolved
@rjbou rjbou added the PR: QUEUED Pending pull request, waiting for other work to be merged or closed label Mar 11, 2021
@rjbou
Copy link
Collaborator

rjbou commented Mar 19, 2021

rebased on 4575

@rjbou rjbou removed the PR: QUEUED Pending pull request, waiting for other work to be merged or closed label Mar 19, 2021
@rjbou
Copy link
Collaborator

rjbou commented Mar 19, 2021

And added some missing commits. Good to go on green CI!

@rjbou
Copy link
Collaborator

rjbou commented Mar 19, 2021

erf, macos readlink error

@dra27
Copy link
Member Author

dra27 commented Mar 19, 2021

@AltGr is on it in #4609 and #4610

@rjbou rjbou merged commit 55c632c into ocaml:master Mar 23, 2021
@rjbou
Copy link
Collaborator

rjbou commented Mar 23, 2021

Thanks!

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

Successfully merging this pull request may close these issues.

opam list --silent should be --check
2 participants