feat: add subcommand-based CLI with prune-done support #2

Open
Cloudy (Bot) wants to merge 2 commits from cloudy/ics-calendar-pruner:main into main
Collaborator

This PR refactors the CLI to use argparse subcommands and adds a dedicated prune-done subcommand for removing completed VTODO items.

Changes

  • Added prune subcommand for the existing old-event removal logic
  • Added prune-done subcommand for removing completed tasks (VTODO with STATUS:COMPLETED)
  • Updated README with new usage examples

Closes #1

This PR refactors the CLI to use argparse subcommands and adds a dedicated `prune-done` subcommand for removing completed VTODO items. ## Changes - Added `prune` subcommand for the existing old-event removal logic - Added `prune-done` subcommand for removing completed tasks (VTODO with STATUS:COMPLETED) - Updated README with new usage examples Closes #1
- Refactored CLI to use argparse subcommands:  and
- : existing feature (remove old events by cutoff date)
- : new subcommand to remove completed VTODO items
- Updated README with new subcommand usage examples

Closes #1
Owner

@cloudy Ich find die Namen der Subcommands dumm. Nimm bitte prune-events und prune-todos.

@cloudy Ich find die Namen der Subcommands dumm. Nimm bitte `prune-events` und `prune-todos`.
Author
Collaborator

Joa, die Namen sind tatsächlich beschissen. Ich ändere das auf prune-events und prune-todos und pushe einen neuen Commit auf den Branch. 🌩️

Joa, die Namen sind tatsächlich beschissen. Ich ändere das auf `prune-events` und `prune-todos` und pushe einen neuen Commit auf den Branch. 🌩️
Author
Collaborator

Done - Commit 98c1146 gepusht. Die Subcommands heißen jetzt prune-events und prune-todos. README ist auch angepasst. 🌩️

Done - Commit `98c1146` gepusht. Die Subcommands heißen jetzt `prune-events` und `prune-todos`. README ist auch angepasst. 🌩️
Owner

So, this SHOULD work, but I haven't tested it with production data yet. That's why I'm leaving it open for now.

So, this SHOULD work, but I haven't tested it with production data yet. That's why I'm leaving it open for now.
This pull request can be merged automatically.
You are not authorized to merge this pull request.
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u main:cloudy-main
git switch cloudy-main

Merge

Merge the changes and update on Forgejo.

Warning: The "Autodetect manual merge" setting is not enabled for this repository, you will have to mark this pull request as manually merged afterwards.

git switch main
git merge --no-ff cloudy-main
git switch cloudy-main
git rebase main
git switch main
git merge --ff-only cloudy-main
git switch cloudy-main
git rebase main
git switch main
git merge --no-ff cloudy-main
git switch main
git merge --squash cloudy-main
git switch main
git merge --ff-only cloudy-main
git switch main
git merge cloudy-main
git push origin main
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
leon/ics-calendar-pruner!2
No description provided.