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

dir-locals: Set some project specific local variables #712

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

pabl0
Copy link
Contributor

@pabl0 pabl0 commented Mar 16, 2025

.dir-locals.el: Set indent-tabs-mode to nil to avoid whitespace problems (particularly with aggressive-indent-mode), as the project predominantly uses spaces for indentation. Ensure sentences end with double spaces to adhere to the house style. Point bug-reference-url-format to Github issues; while there are few bug references in the codebase, this may be useful later.

.dir-locals.el: Set `indent-tabs-mode' to nil to avoid whitespace
problems (particularly with `aggressive-indent-mode'), as the project
predominantly uses spaces for indentation.  Ensure sentences end with
double spaces to adhere to the house style.  Point
`bug-reference-url-format' to Github issues; while there are few bug
references in the codebase, this may be useful later.
@benthamite
Copy link
Contributor

Perhaps we could also set fill-column to its default value of 70? Some users (myself included) have a different value set, which means that unless they always remember to call set-fill-column before filling a paragraph, the file will not be filled consistently.

@pabl0
Copy link
Contributor Author

pabl0 commented Mar 27, 2025

The Emacs project .dir-ignore has 72 for both fill-column and emacs-lisp-docstring-fill-column (which is also default in Emacs 30). This change was made in February 2024 (72b1379f) and, in my opinion, is well justified.. @karthink what do you think?

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