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

Don't require sudo if we can already write to target(s) #29

Closed
asgrim opened this issue Sep 4, 2024 · 0 comments · Fixed by #33
Closed

Don't require sudo if we can already write to target(s) #29

asgrim opened this issue Sep 4, 2024 · 0 comments · Fixed by #33
Assignees
@asgrim
Labels
enhancement New feature or request
Milestone
0.1.0

Comments

@asgrim
Copy link
Collaborator

asgrim commented Sep 4, 2024

At the moment, pie install will ask for sudo regardless, even if we can already write to the target path(s).
We should improve this to check if we actually NEED to even ask.
@asgrim asgrim added the enhancement New feature or request label Sep 4, 2024
@asgrim asgrim self-assigned this Sep 4, 2024
@asgrim asgrim mentioned this issue Sep 10, 2024
Merged
@asgrim asgrim added this to the 0.1.0 milestone Dec 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@asgrim asgrim

Labels
enhancement New feature or request
Projects
None yet
Milestone
0.1.0
Development

Successfully merging a pull request may close this issue.

Only use sudo if needed asgrim/pie
1 participant
@asgrim