Skip to content

chore: sync bindings in README, config and code - #788

Merged
RobertMueller2 merged 1 commit into
Satty-org:mainfrom
robertwidfen:chore/bindings-doc
Oct 28, 2026
Merged

RobertMueller2 merged 1 commit into
Satty-org:mainfrom
robertwidfen:chore/bindings-doc

Conversation

@robertwidfen

Copy link
Copy Markdown
Collaborator

Description

They were not in sync.

Question actions_on_enter - should we follow:

  • the README list save to clipboard as default
  • the code has no actions assigned by default

Checklist

  • I've read CONTRIBUTING.md and AI_POLICY.md
  • all commits compile
  • PR title uses (conventional commit message)[https://www.conventionalcommits.org/]
  • includes breaking changes - please try to avoid if possible
    • I added "!" for attention in the PR title
    • I filled out "Breaking change" section above
  • any merge conflicts were resolved via rebase rather than merge
  • includes LLM/Generative AI generated code
    • I understand the code on a technical level
    • I vouch for the code

@robertwidfen
robertwidfen requested a review from a team October 26, 2026 08:53
@robertwidfen

Copy link
Copy Markdown
Collaborator Author

Please do not miss:

Question actions_on_enter - should we follow:

  • the README lists save to clipboard as default
  • the code has no actions assigned by default

Currently I added it in the code - which is a breaking change. 🤔

@RobertMueller2

Copy link
Copy Markdown
Member

Please do not miss:

Question actions_on_enter - should we follow:

* the README lists save to clipboard as default

* the code has no actions assigned by default

Currently I added it in the code - which is a breaking change. 🤔

I have not missed it ;) I was too lazy to type while on mobile earlier. It's definitely justifiable to add it if the documentation says it's there. Since nobody has complained about it, it's just as justifiable to remove it from the documentation. Believe it or not,I don't think I have an opinion on this. ;)

@robertwidfen

Copy link
Copy Markdown
Collaborator Author

Ok, then we keep it like this - IMHO it does no harm - if the documented behavior would have been "exit" I would have decided the other way around ;-)

So let's merge it now.

@RobertMueller2
RobertMueller2 merged commit 99127c8 into Satty-org:main Oct 28, 2026
4 checks passed
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