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

replace with same editorconfig as used in gutenberg #297

Merged
merged 1 commit into from
Apr 21, 2023

Conversation

erikyo
Copy link
Collaborator

@erikyo erikyo commented Apr 20, 2023

What?

I realised that the editorconfig was changed but that configuration always gave me troubles with js/webstorm, and may lead to a worse development experience

Why?

tab_width = 4 is forcing the ide to use this setting whereas if you do not set anything, you have the option of using your preference from the ide.
Fixes Yaml files that should not use tabs

How?

Copy and pasted the gutenberg config file that is working fine https://github.com/WordPress/gutenberg/blob/trunk/.editorconfig

Copy link
Collaborator

@johnhooks johnhooks left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I agree with this change. I didn't initially notice because I already have VSCode set to display tabs as 2 spaces which overrides the setting in the editor config. Though tabs, spaces and display width can be a contentious subject. We have chooses tabs but should leave display width to personal preference.

@erikyo erikyo merged commit 5d05860 into WordPress:develop Apr 21, 2023
@erikyo erikyo deleted the lint/editor_config branch April 21, 2023 09:41
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