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

Improve CLI interactions and caching URL #1320

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

elcritch
Copy link
Contributor

Some more work on CLI and some related changes for speed.

  • suppress redundant info like installed package foo on default outputs
  • adds --info flag to print the extra info without a full verbose (MediumPriority)
  • modifies nimble deps to print out only direct dependencies and versions
    • this seems more like what people expect when running nimble deps
  • nimble deps --tree prints out the full dedendency tree
  • info about url based deps are cached in a packages_nimblecached.json PkgList

I did some simple timing on one of my projects and get:

# Before cachine URL checks
figuro> nimble deps  2.03s user 0.55s system 26% cpu 9.877 total
# After caching URL checks
figuro> nimble deps  1.38s user 0.24s system 83% cpu 1.931 total

@elcritch
Copy link
Contributor Author

elcritch commented Dec 21, 2024

I'd also like to add a nimble deps --update and maybe a nimble deps --install options. Ideally a nimble deps --update would be a great way to do network related work but upon explicit request. This is how Elixir's mix package manager works and it's great to use.

@Araq what would you think about adding that?

@Araq
Copy link
Member

Araq commented Dec 21, 2024

@Araq what would you think about adding that?

Fine with me but the naming could be better.

@elcritch elcritch force-pushed the cli-ux-improvements branch from 0a73e14 to f007e0b Compare January 9, 2025 04:34
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.

2 participants