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

Run opam init for a fresh install #5501

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

Conversation

sabine
Copy link

@sabine sabine commented Apr 3, 2023

This PR extends #5500 by adding a call to opam init in the case of a fresh install.

Before, only in the case of a non-fresh install, opam init --reinit -ni was called.

@kit-ty-kate kit-ty-kate added the PR: QUEUED Pending pull request, waiting for other work to be merged or closed label Apr 3, 2023
@sabine sabine force-pushed the opam_init_if_fresh branch from 6c79e40 to 5766129 Compare April 3, 2023 18:02
@sabine
Copy link
Author

sabine commented Apr 4, 2023

I'm not 100% sure this is the right way to go about it. An alternative would be to prompt Run opam init now? [Y/n].

The intent behind this is to make installing opam and setting it up more obvious/faster for newcomers by railroading them through the obvious steps.

@sabine sabine force-pushed the opam_init_if_fresh branch 2 times, most recently from d5658dd to 3889d9d Compare April 5, 2023 16:23
@sabine sabine force-pushed the opam_init_if_fresh branch from 3889d9d to 2107cb2 Compare April 5, 2023 16:43
@sabine sabine force-pushed the opam_init_if_fresh branch from 2107cb2 to ce9b0b9 Compare April 5, 2023 17:07
@kit-ty-kate kit-ty-kate added this to the 2.4.0~alpha1 milestone Sep 25, 2024
@kit-ty-kate
Copy link
Member

An alternative would be to prompt Run opam init now? [Y/n].

i'd much prefer this solution. I'll do that once #5500 is merged

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PR: QUEUED Pending pull request, waiting for other work to be merged or closed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants