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

tweak docs to clarify install depsOnly option #1020

Merged
merged 4 commits into from
Aug 16, 2022

Conversation

elcritch
Copy link
Contributor

So I never realized you could leave out the pkgname and it'd use the local nimble file. It's clear in hindsight but reading the docs and help I always assumed install always required the package name. Hopefully this tweak might save others from missing this too.

Copy link
Collaborator

@dom96 dom96 left a comment

Choose a reason for hiding this comment

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

Nice, thanks. I think it would be good to have this after the paragraph starting "If you don't specify a parameter..." can you move it there?

@elcritch
Copy link
Contributor Author

Certainly, I'll update it.

@elcritch
Copy link
Contributor Author

I started editing but wasn't quite happy with it. Adding the subsection seemed to make it a bit more clearer to the different usages of install (package manager vs local package dev).

Copy link
Collaborator

@dom96 dom96 left a comment

Choose a reason for hiding this comment

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

Thanks!

@dom96 dom96 merged commit 05ecb8e into nim-lang:master Aug 16, 2022
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.

None yet

2 participants