ObsidianIRC: Update capabilities in sw_clients.yml #567
Conversation
There was a problem hiding this comment.
Pull Request Overview
Adds additional declared stable capabilities for the ObsidianIRC client in the sw_clients.yml data file. Expands the capabilities list with multiple IRCv3 (and related) features.
- Adds numerous capability keys (e.g., account-notify, batch, chathistory, multiline, standard-replies)
- Extends the stable capability set for the client in YAML metadata
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
| cap-notify: | ||
| account-notify: | ||
| account-tag: | ||
| account-registration: | ||
| away-notify: | ||
| batch: | ||
| bot-mode: | ||
| channel-rename: | ||
| chathistory: | ||
| echo-message: | ||
| extended-join: | ||
| message-redaction: | ||
| message-tags: | ||
| msgid: | ||
| multiline: | ||
| multi-prefix: | ||
| react-client-tag: | ||
| sasl-3.1: | ||
| server-time: | ||
| setname: | ||
| standard-replies: | ||
| userhost-in-names: |
There was a problem hiding this comment.
[nitpick] The newly added capability entries are not in a consistent ordering (e.g., multi-prefix appears after multiline, and related capability types are not grouped), which makes future diffs harder to review. Consider alphabetizing the block or following an established grouping convention; for example: account-notify, account-registration, account-tag, away-notify, batch, bot-mode, cap-notify, channel-rename, chathistory, echo-message, extended-join, message-redaction, message-tags, msgid, multi-prefix, multiline, react-client-tag, sasl-3.1, server-time, setname, standard-replies, userhost-in-names.
2 participants
No description provided.