.editorconfig - Issue 29346533: Issue 3693 - Created global editorconfig

Side by Side Diff

Use n/p to move between diff chunks; N/P to move between comments.

Keyboard Shortcuts

	File
u :	up to issue
m :	publish + mail comments
M :	edit review message
j / k :	jump to file after / before current file
J / K :	jump to next file with a comment after / before current file
	Side-by-side diff
i :	toggle intra-line diffs
e :	expand all comments
c :	collapse all comments
s :	toggle showing all comments
n / p :	next / previous diff chunk or comment
N / P :	next / previous comment
<Up> / <Down> :	next / previous line
<Enter> :	respond to / edit current comment
d :	mark current comment as done

	Issue
u :	up to list of issues
m :	publish + mail comments
j / k :	jump to patch after / before current patch
o / <Enter> :	open current patch in side-by-side view
i :	open current patch in unified diff view

	Issue List
j / k :	jump to issue after / before current issue
o / <Enter> :	open current issue
# :	close issue

	Comment/message editing
<Ctrl> + s or <Ctrl> + Enter :	save comment
<Esc> :	cancel edit

Side by Side Diff: .editorconfig

Issue 29346533: Issue 3693 - Created global editorconfig (Closed)

Patch Set: Created June 15, 2016, 1:16 p.m.

Left:
Right:

Use n/p to move between diff chunks; N/P to move between comments.

Jump to:

View unified diff | Download patch

OLD	NEW
(Empty)
	1 # EditorConfig is awesome: http://EditorConfig.org

	2

	3 # top-most EditorConfig file

	4 root = true

	5

	6 # Defaults
	kzar 2016/09/16 10:01:05 Perhaps we should add `max_line_length = 80`? Perhaps we should add `max_line_length = 80`? Wladimir Palant 2016/09/16 10:29:01 "Supported By A Limited Number of Editors" But ev Show quoted text On 2016/09/16 10:01:05, kzar wrote: > Perhaps we should add `max_line_length = 80`? "Supported By A Limited Number of Editors" But even if it were supported widely, I don't really see how hard wrapping would be useful. kzar 2016/09/16 10:37:51 Ah you're right, never mind. Show quoted text On 2016/09/16 10:29:01, Wladimir Palant wrote: > On 2016/09/16 10:01:05, kzar wrote: > > Perhaps we should add `max_line_length = 80`? > > "Supported By A Limited Number of Editors" > > But even if it were supported widely, I don't really see how hard wrapping would > be useful. Ah you're right, never mind. Thomas Greiner 2016/09/16 11:55:54 According to https://github.com/editorconfig/edito Show quoted text On 2016/09/16 10:29:01, Wladimir Palant wrote: > On 2016/09/16 10:01:05, kzar wrote: > > Perhaps we should add `max_line_length = 80`? > > "Supported By A Limited Number of Editors" > > But even if it were supported widely, I don't really see how hard wrapping would > be useful. According to https://github.com/editorconfig/editorconfig/wiki/Property-research:-maximum-... the behavior of this is inconsistent anyway. In Sublime Text and Vim this is merely adding an indicator (which is also why I'm using it) whereas in Emacs it wraps the lines automatically. Having an indicator is still quite useful though.
	7 indent_style = space

	8 indent_size = 2

	9 end_of_line = lf

	10 charset = utf-8

	11 trim_trailing_whitespace = true
	Thomas Greiner 2016/09/16 11:55:54 For me personally, this one is quite inconvenient For me personally, this one is quite inconvenient because I save my files quite often while editing. Therefore it would cause my cursor to jump to the beginning of the line if I had this setting enabled. I assume that I'm probably the only one with that issue and that the config can still be overridden but just wanted to note it. Wladimir Palant 2016/09/16 12:15:32 I guess that's why Atom isn't trimming whitespace Show quoted text On 2016/09/16 11:55:54, Thomas Greiner wrote: > For me personally, this one is quite inconvenient because I save my files quite > often while editing. Therefore it would cause my cursor to jump to the beginning > of the line if I had this setting enabled. I guess that's why Atom isn't trimming whitespace on the current line - for people like you ;)
	12 insert_final_newline = true

	13

	14 # PEP 8

	15 [*.py]

	16 indent_size = 4

OLD	NEW

« no previous file with comments | « no previous file | no next file » | no next file with comments »