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

feat: implemented delete #818

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

feat: implemented delete #818

wants to merge 2 commits into from

Conversation

ninaagrawal
Copy link
Contributor

Implemented the delete function. Did not test implement actually deleting the files for safety

Signed-off-by: Nina 'Nino' Agrawal <[email protected]>
@alilleybrinker
Copy link
Collaborator

Make sure to test, it should be safe to delete plugins since on a re-run they should be redownloaded, and if they're not that's a bug we don't want to ship to users.

Also don't leave commented-out code in. If it's not used, delete it. If this PR is still a draft, make sure to mark it as a draft.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Todo
Development

Successfully merging this pull request may close these issues.

3 participants