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

Documentation, contribution and code style guidelines #10

Open
AndersonTorres opened this issue Oct 5, 2022 · 3 comments
Open

Documentation, contribution and code style guidelines #10

AndersonTorres opened this issue Oct 5, 2022 · 3 comments

Comments

@AndersonTorres
Copy link
Member

AndersonTorres commented Oct 5, 2022

We need some form of code style and contribution guidelines in general.

Edit 2022-11-07:

Some useful links

https://github.com/alphapapa/emacs-package-dev-handbook

https://github.com/bbatsov/emacs-lisp-style-guide

https://gnu.org/software/emacs/manual/html_node/elisp/Tips.html

Another good thing to do is study and reuse the code of GNU APL Mode

Edit 2022-11-20:

Take notes on guidelines for MELPA compliance

Edit 2023-02-14

I am interested in using conventional commits, but with my own twist: I don't like abbreviations as "feat", "chore" etc.

https://www.conventionalcommits.org/en/v1.0.0/

@AndersonTorres AndersonTorres changed the title Contribute guidelines and code style Documentation - including guidelines to contribution and code style Apr 2, 2023
@ilohmar
Copy link
Contributor

ilohmar commented Apr 12, 2023

This issue seems to be focused on formal processes and "best practices". I have a more general question regarding the development of this package: It seems to me to break a lot of emacs conventions, some minor issues, some with broader consequences. This is very common in language-mode packages, because the authors are often experts with a lot of experience in environments that are quite different from Emacs.

My natural reaction is to go ahead and just change a lot of stuff, but that precludes any possibility of merging things later.

Is there interest to bring this package more in line with Emacs conventions (including, but not limited to, drastically reducing the number of files, not changing global Emacs settings, using more standard facilities for keymaps, dropping "-face" suffixes etc)?

The reason why I am asking is that

  1. directly preparing such intrusive changes as PRs is tedious, and it can easily appear hostile to the people who put work and effort into the package, which is useful as it is.

  2. I have absolutely no problem maintaining my own mode, but I do not want to be irreverent to the existing one.

Any hints are appreciated!

EDIT: After browsing the commit history some more, I realize that many of the things I would try to change were deliberate decisions. I surely do not want to interfere with that, so (barring other developments) I will simply maintain a separate mode.

@AndersonTorres
Copy link
Member Author

Disclaimer: I am far from being an Emacs expert.

Is there interest to bring this package more in line with Emacs conventions (including, but not limited to, drastically reducing the number of files, not changing global Emacs settings, using more standard facilities for keymaps, dropping "-face" suffixes etc)?

Yes it is. The more it integrates with Emacs, the better.

The reason why I am asking is that directly preparing such intrusive changes as PRs is tedious, and it can easily appear hostile to the people who put work and effort into the package, which is useful as it is.

In the worst case I can just do a huge revert :) Bring your ideas on.

I have absolutely no problem maintaining my own mode, but I do not want to be irreverent to the existing one.

I am interested in not to split the community. If there is a competition between bqn-modes, it becames harder to justify one mode above the other, and I would just close the repo and contribute with the newer one.

EDIT: After browsing the commit history some more, I realize that many of the things I would try to change were deliberate decisions.

More or less. Many of them made sense on that time, however there are other than I would easily regret.

@ilohmar
Copy link
Contributor

ilohmar commented Apr 13, 2023

Ok great, then I'll start with the most intrusive one. I will create a PR merging most files later today.

@AndersonTorres AndersonTorres changed the title Documentation - including guidelines to contribution and code style Documentation, contribution and code style guidelines Apr 16, 2023
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