Differential D15639

Add editorconfig
AbandonedPublic
Actions

Authored by tcanabrava on Sep 21 2018, 9:05 AM.

Details

Reviewers
apol
gszymaszek
Summary

tabs / spaces are project specific configuration but the
majority of the editors this is a global option that doesn't
depend of the project, and it's extremely annoying to change
the settings for every project that you open.

With editorconfig we have this ready and automated for +10 editors
including kate and variants, qt creator, atom, visual studio code,
etc.

Diff Detail

Repository
R324 Blinken
Branch
addEditorConfig
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 3058
Build 3076: arc lint + arc unit
tcanabrava created this revision.Sep 21 2018, 9:05 AM
Restricted Application added a project: KDE Edu. · View Herald TranscriptSep 21 2018, 9:05 AM
Restricted Application added a subscriber: kde-edu. · View Herald Transcript
tcanabrava requested review of this revision.Sep 21 2018, 9:05 AM
apol added a subscriber: apol.Sep 21 2018, 11:21 AM
apol added inline comments.
.editorconfig
16

Do you really need to set an indent size for every time if it's already set for [*]?

24

Why false here?

tcanabrava added inline comments.Sep 21 2018, 11:27 AM
.editorconfig
16

copy paste issue.

24

some blocks on md files need trailing whitespaces to be displayed correctly

apol requested changes to this revision.Sep 21 2018, 2:01 PM

fix it

This revision now requires changes to proceed.Sep 21 2018, 2:01 PM
gszymaszek requested changes to this revision.Sep 21 2018, 7:05 PM
gszymaszek added a subscriber: gszymaszek.
gszymaszek added inline comments.
.editorconfig
3

Is there any known max_line_length?

4–9

Could you sort it?

11–21

Not needed IMO.

tcanabrava abandoned this revision.Nov 21 2019, 1:09 PM

Revision Contents
Changeset List

PathPackages
AM.editorconfig (24 lines)

Diff 42038

View Options

.editorconfig

Loading...