* editorconfig
@ 2024-03-22 18:17 Toomas Soome
2024-03-22 19:39 ` [developer] editorconfig Bill Sommerfeld
0 siblings, 1 reply; 2+ messages in thread
From: Toomas Soome @ 2024-03-22 18:17 UTC (permalink / raw)
To: illumos-developer
[-- Attachment #1: Type: text/plain, Size: 440 bytes --]
hi!
In light of #15505, do we want to add .editorconfig into root of illumos-gate? I have using this for 4 years now (and the same is in use in OpenZFS):
=====================================
root = true
[*]
end_of_line = lf
insert_final_newline = true
trim_trailing_whitespace = true
[*.{c,h}]
tab_width = 8
indent_style = tab
=====================================
See also: https://editorconfig.org/
rgds,
toomas
[-- Attachment #2: Type: text/html, Size: 8401 bytes --]
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [developer] editorconfig
2024-03-22 18:17 editorconfig Toomas Soome
@ 2024-03-22 19:39 ` Bill Sommerfeld
0 siblings, 0 replies; 2+ messages in thread
From: Bill Sommerfeld @ 2024-03-22 19:39 UTC (permalink / raw)
To: developer
On 3/22/24 11:17, Toomas Soome via illumos-developer wrote:
> trim_trailing_whitespace = true
I'd be wary of unintended consequences from this one being applied too
broadly.
One case I'm aware of - there are more than a few *.out files in
usr/src/test/util-tests/tests/dis/*/*.out that have trailing whitespace,
because they contain expected dis(1) output.
Aside from that, I think it's generally a good idea.
- Bill
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2024-03-22 19:40 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2024-03-22 18:17 editorconfig Toomas Soome
2024-03-22 19:39 ` [developer] editorconfig Bill Sommerfeld
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).