feat: add edit --priority and edit --list options - #5
Merged
Merged
Conversation
Adapted from the priority portion of upstream pull request keith/reminders-cli#87 by Aaron Fields; the due date portion of that pull request is already covered by our existing --due-date and --clear-due-date support. Reuses the existing Priority enum instead of duplicating its raw value mapping. Co-authored-by: Aaron Fields <spirotot@gmail.com>
Lets a reminder be moved between lists in place instead of requiring delete-and-re-add, by reassigning its EKCalendar via the same calendar(withName:) lookup already used to resolve the source list.
udondan
added a commit
that referenced
this pull request
Sep 13, 2026
feat: add edit --priority and edit --list options
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
edit --priority/--clear-priorityto set or clear a reminder's priority. This is the priority portion of upstream pull request Add the ability to edit or clear the due date, and priority. keith/reminders-cli#87 by Aaron Fields (@bitfieldz); the due date portion of that PR is already covered by our existing--due-date/--clear-due-datesupport. Adapted to reuse the existingPriorityenum instead of duplicating its raw value mapping, and committed with Aaron Fields preserved as author plus aCo-authored-bytrailer.edit --list <name>to move a reminder to a different list in place, by reassigning its calendar via the existingcalendar(withName:)lookup.Test plan
swift build -Xswiftc -warnings-as-errorsswift test -Xswiftc -warnings-as-errors(52 tests passing)swift run remindersagainst a scratch list:edit <list> <index> --priority high/--clear-priority--priority+--clear-prioritytogether correctly errorsedit <list> <index>with no options correctly errorsedit <list> <index> --list <other-list>moves the reminder and it shows up under the other list--list <bogus-list>correctly errors with "No reminders list matching ..."