-
Notifications
You must be signed in to change notification settings - Fork 116
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
add short section on contributing to README #5
Comments
or as a separate CONTRIBUTING.(org|md) file |
I wrote a first draft, feel free to give feedback or to improve it :) |
Making a pattern of modules would be nice to have everything cohesive, so newcomers know how to create new modules. E.g: ;;; rational-<module-name>.el --- <Little description> -*- lexical-binding: t; -*-
< Start module >
(provide 'rational-<module-name>) And maybe an org file with the module's documentation. |
@mfxarc A section on "Creating new modules" would be nice! Org files for module documentation would also be great, but it'd be nice if we could generate those from the source of the modules so that we wouldn't have to keep things up to date in 2 places. |
Wouldn't it be better if the pattern was how Emacs (and elisp mode) like to have descriptions in files? Or even have a yasnippet or something like that added, with this as a snippet? |
We should definitely have expanded descriptions in the source files, just haven't gotten to that yet. I think we could add a Skeleton (or some other form of Emacs auto-insert) to create a new file template that can be called up from within this configuration: https://www.gnu.org/software/emacs/manual/html_node/autotype/Autoinserting.html |
The Principles links follows to page not found(In the contributing section) |
Yeah I noticed this this morning as well. Kind of annoying because it's a valid org link :( |
Also, the second little elisp-style button isn't working somehow. Perhaps because there's a space ( |
Thanks @japhir! |
as discussed in the stream, would be nice to have a section on how to contribute.
The text was updated successfully, but these errors were encountered: