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

rv info #135

Merged
merged 9 commits into from
Mar 5, 2025
Merged

rv info #135

merged 9 commits into from
Mar 5, 2025

Conversation

wes-a2ai
Copy link
Contributor

@wes-a2ai wes-a2ai commented Mar 3, 2025

rv info. Listing the package sources, installation summary, and the packages available in the repositories.

Additional iteration to come

@wes-a2ai wes-a2ai requested a review from Keats March 3, 2025 16:49
@wes-a2ai
Copy link
Contributor Author

wes-a2ai commented Mar 3, 2025

Example output:

OS: linux (x86_64)
R Version: 4.4

== Dependencies == 
Library: example_projects/simple/rv/library/4.4/x86_64/jammy
Installed: 0/31

Package Sources: 
  posit: 0/29 binary packages, 0/2 source packages

Installation Summary: 
  posit: 7/31 in cache (1 to compile), 23/31 to download (1 to compile)

== Remote == 
posit (https://packagemanager.posit.co/cran/2024-12-16): 21810 binary packages, 21810 source packages

@dpastoor
Copy link
Member

dpastoor commented Mar 3, 2025

I think this is "good enough" for an initial version (i'm quite happy with it already!). IMO there is going to be a lot more tweaking we'll want to mess with, but I want to at least get the initial release out the door now and this sets us up nicely.

to copy-paste part of the slack thread, where I see this going is a deliniation in the defaults and flag constructs between info/plan such that

rv plan --> what is about to happen when you run sync

rv info --> tell you about the state of your project and system and how it compares to the lockfile/project

@dpastoor
Copy link
Member

dpastoor commented Mar 5, 2025

I just tried info on a real project - as mentioned above, i think we need to iterate more on the output but i'm happy with this enough that we should get this out!

@dpastoor dpastoor merged commit 5bb80dd into main Mar 5, 2025
6 checks passed
@dpastoor dpastoor deleted the info branch March 5, 2025 13:24
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