Skip to content

Add --nodeclined support to remind#636

Draft
portothree wants to merge 3 commits intoinsanum:mainfrom
portothree:master
Draft

Add --nodeclined support to remind#636
portothree wants to merge 3 commits intoinsanum:mainfrom
portothree:master

Conversation

@portothree
Copy link
Copy Markdown

No description provided.

@dbarnett
Copy link
Copy Markdown
Collaborator

Looks like the remind command still doesn't support a --nodeclined option, and should, but it doesn't look like this change actually finishes plumbing it through. LMK if you still plan to work on that or would prefer someone else pick it up.

snomiao added a commit to snomiao/gcalcli that referenced this pull request Feb 17, 2026
…#636)

Wire output_parser (which includes --nodeclined/ignore_declined) into the
remind subparser and apply the existing _DeclinedEvent() filter pattern
inside Remind() to skip declined events.

The original PR only added a dead parameter; this implementation uses the
existing self.options['ignore_declined'] pattern consistent with all other
commands that respect --nodeclined.

Co-Authored-By: Gustavo Porto <gustavo@portosanti.com>
Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
snomiao added a commit to snomiao/gcalcli that referenced this pull request Feb 17, 2026
…#636) (#9)

Wire output_parser (which includes --nodeclined/ignore_declined) into the
remind subparser and apply the existing _DeclinedEvent() filter pattern
inside Remind() to skip declined events.

The original PR only added a dead parameter; this implementation uses the
existing self.options['ignore_declined'] pattern consistent with all other
commands that respect --nodeclined.

Co-authored-by: Gustavo Porto <gustavo@portosanti.com>
Co-authored-by: Claude Sonnet 4.5 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants