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.

Sign up for GitHub

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

Jump to bottom

Create tutorial: Running Commands in an Opam Switch #1825

Merged
merged 4 commits into from
Jan 8, 2024
Merged

Create tutorial: Running Commands in an Opam Switch #1825

merged 4 commits into from
Jan 8, 2024

Conversation

cuihtlauac
Copy link
Collaborator

Fix: #1824

sabine
sabine reviewed Dec 6, 2023
View reviewed changes
Copy link
Collaborator

@sabine sabine left a comment

Choose a reason for hiding this comment

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

I think it needs a bit more context which switch we're setting up there. I.e. closest local switch from the current working directory, or active global switch?

So useful, though!

christinerose
christinerose reviewed Dec 6, 2023
View reviewed changes
Copy link
Collaborator

@christinerose christinerose left a comment

Choose a reason for hiding this comment

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

Found an incomplete sentence
Also, changed formatting of opam to match style guide. All lowercase except at the beginning of a sentence.

data/tutorials/platform/bp_09_opam_path.md Outdated
```bash
eval $(opam env)
```
After running `direnv allow`, `direnv` will handle the opam switch activation for you whenever you.
Copy link
Collaborator

Choose a reason for hiding this comment

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

This appears to be an incomplete sentence.

@cuihtlauac cuihtlauac added documentation new-doc Track creation of new content for the OCaml Documentation labels Dec 8, 2023
@sabine sabine merged commit 9c259b1 into main Jan 8, 2024
2 of 3 checks passed
@sabine sabine deleted the doc-opam-path branch January 8, 2024 16:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@christinerose christinerose christinerose left review comments

@sabine sabine sabine approved these changes

Assignees
No one assigned
Labels
documentation new-doc Track creation of new content for the OCaml Documentation
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Create Doc on opam env, opam exec or direnv
3 participants
@cuihtlauac @sabine @christinerose