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

Feat: Implement cargo nexus clean #70

Open
danielmarinq opened this issue Feb 16, 2024 · 2 comments
Open

Feat: Implement cargo nexus clean #70

danielmarinq opened this issue Feb 16, 2024 · 2 comments
Assignees
@slumber
Labels
enhancement New feature or request
Milestone
v1

Comments

@danielmarinq
Copy link
Contributor

The clean command should delete all public parameter files and run cargo clean under the hood to remove compilation targets as well.
@danielmarinq danielmarinq added the enhancement New feature or request label Feb 16, 2024
@danielmarinq danielmarinq added this to the v1 milestone Feb 16, 2024
@slumber
Copy link
Contributor

slumber commented Feb 16, 2024

prove and pp commands use . path unless explicitly specified by the user. And unlike dev cli the cargo command isn't tied to any package or workspace. What would clean command do then? The issue doesn't really make sense to me.

@danielmarinq
Copy link
Contributor Author

Discussed today. TL;DR: @slumber shall decide where to best place the pps.

@govereau govereau removed their assignment Feb 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@slumber slumber

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

No branches or pull requests
3 participants
@govereau @slumber @danielmarinq