Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Naming convention inconsistency #409

Open
raffaem opened this issue Jan 13, 2024 · 1 comment
Open

Naming convention inconsistency #409

raffaem opened this issue Jan 13, 2024 · 1 comment

Comments

@raffaem
Copy link
Contributor

raffaem commented Jan 13, 2024

  1. Why the function to send a line is called SlimeLineSend (object+verb), while the one to send a cell is called SlimeSendCell (verb+object)?
  2. Where are those functions documented? Even in Advanced configuration I did not find SlimeLineSend
@raffaem raffaem changed the title Inconsistency Naming convention inconsistency Jan 13, 2024
@jpalardy
Copy link
Owner

Hi @raffaem

For 1, there's no specific reason. Many contributions, over time, by different people, led to this drift. At this point however, changing it might break people's configuration.

For 2 — it's a similar take. Initially, the "public API" of the plugin were the key bindings. While helper functions were added over time, it wasn't always clear how/where to document.

If you want to take a stab, please send a PR.

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

No branches or pull requests

2 participants