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

Configure .editorconfig for IDE #339

Merged
merged 3 commits into from Apr 24, 2015
Merged

Configure .editorconfig for IDE #339

merged 3 commits into from Apr 24, 2015

Conversation

shivapoudel
Copy link
Contributor

@GaryJones this not work very well:

[{*.md,*.yml}]

Usually works with full names like:

[{README.md,.coveralls.yml,.etc.yml}]

I'm using with VIM and Sublime and does not work. Impossible to use something unstable only for satisfaction of saving three lines in a configuration file.
;)

@GaryJones
Copy link
Member

I'm only going by what I see on http://develop.svn.wordpress.org/trunk/.editorconfig .

Try [*.{md,yml}] instead, as that's the example given on http://editorconfig.org/

@GaryJones GaryJones added this to the 2.5.0 milestone Apr 24, 2015
GaryJones added a commit that referenced this pull request Apr 24, 2015
Configure .editorconfig for IDE
@GaryJones GaryJones merged commit f30e0e6 into TGMPA:develop Apr 24, 2015
@GaryJones
Copy link
Member

Thanks @shivapoudel!

@shivapoudel
Copy link
Contributor Author

Markdown trim_trailing_whitespace should be false xD

For instance: We have Lead Developer and there at the end we have added the trailing whitespace for newline trick ;)

@shivapoudel shivapoudel deleted the editorconfig branch April 24, 2015 22:26
@GaryJones
Copy link
Member

I know. My ST3 is already setup with custom rules for .md files :-)

When I commented, it was against a line that said no newline at the end of the file, not trimming whitespace at the end of the line.

@shivapoudel
Copy link
Contributor Author

xD

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants