Add an optional id to dunstctl close

[bynect]

No description provided.

[codecov-commenter]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 65.95%. Comparing base (9fb0216) to head (14ecba5).

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files

@@           Coverage Diff           @@
##           master    #1351   +/-   ##
=======================================
  Coverage   65.95%   65.95%           
=======================================
  Files          50       50           
  Lines        8209     8209           
  Branches      962      962           
=======================================
  Hits         5414     5414           
  Misses       2795     2795           

Flag	Coverage Δ
unittests	65.95% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[zappolowski]

Looks good. It would be nice to adjust the completions as well (at least the summary strings) for zsh and fish as they're misleading otherwise.

[bynect]

Looks good. It would be nice to adjust the completions as well (at least the summary strings) for zsh and fish as they're misleading otherwise.

right, forgot about that 👍🏻

[bynect]

@zappolowski I fixed some typos and updated the completions. Unfortunately we can't autocomplete the id (see my suggestion in #1348).

[bynect]

done.

unrelated question, should we add a completion for dunstify later?

[zappolowski]

unrelated question, should we add a completion for dunstify later?

Yes, this would make sense. I just added it for fish as that's what I'm using.