Skip to content

Add drop rm to remove installed apps - #117

Merged
miniprow[bot] merged 1 commit into
mainfrom
rm
Oct 28, 2026
Merged

miniprow[bot] merged 1 commit into
mainfrom
rm

Conversation

@puerco

@puerco puerco commented Oct 28, 2026

Copy link
Copy Markdown
Member

The inventory could be created, read and updated but not deleted from. drop rm uninstalls apps installed with drop: it deletes the binary or removes the package through the package manager (with sudo when needed) and drops the app from the inventory once that succeeded. Apps are named as drop list shows them, or by repository to remove every app installed from it; a name shared across repositories must be qualified. The command lists what will be removed and asks for confirmation unless --yes is given.

The removal step the installer used to clean up after a reinstall is reused, and its events now say whether they concern a previous installation so the progress output reads right in both cases.

The inventory could be created, read and updated but not deleted from.
drop rm uninstalls apps installed with drop: it deletes the binary or
removes the package through the package manager (with sudo when needed)
and drops the app from the inventory once that succeeded. Apps are named
as drop list shows them, or by repository to remove every app installed
from it; a name shared across repositories must be qualified. The
command lists what will be removed and asks for confirmation unless
--yes is given.

The removal step the installer used to clean up after a reinstall is
reused, and its events now say whether they concern a previous
installation so the progress output reads right in both cases.

Signed-off-by: Adolfo García Veytia (Puerco) <puerco@carabiner.dev>
@miniprow miniprow Bot added lgtm Indicates that a PR is ready to be merged (Looks Good To Me) approved Indicates a PR has been approved by an approver from all required OWNERS files labels Oct 28, 2026
@miniprow

miniprow Bot commented Oct 28, 2026

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: puerco

The full list of commands accepted by this bot can be found here.

Details Approved by a repository administrator.

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@miniprow
miniprow Bot merged commit c1c03cf into main Oct 28, 2026
8 checks passed
@miniprow
miniprow Bot deleted the rm branch October 28, 2026 21:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files lgtm Indicates that a PR is ready to be merged (Looks Good To Me)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant