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

Add .editorconfig file #491

Merged
merged 3 commits into from
Jun 18, 2021
Merged

Add .editorconfig file #491

merged 3 commits into from
Jun 18, 2021

Conversation

schlessera
Copy link
Member

This PR adds an .editorconfig file to let the IDE know how to handle newlines, tabs/spaces, etc. for the project.

This avoids commits like this one, as the IDE will automatically deal with this: 228ec26

@schlessera schlessera added this to the 2.0.0 milestone Jun 18, 2021
@schlessera schlessera requested a review from jrfnl June 18, 2021 06:58
Copy link
Member

@jrfnl jrfnl left a comment

Choose a reason for hiding this comment

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

I'm happy to merge this if it helps people. Not fussed myself as my editor doesn't use the file.

Review Notes:

Not all existing (yml, json) files currently comply with these rules.

Feels to me those should be brought into sync in conjunction with this PR.

I haven't checked the PHP files, but generally speaking that's covered by the CS rules and checked anyhow.

Copy link
Member

@jrfnl jrfnl left a comment

Choose a reason for hiding this comment

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

:shipit:

@jrfnl jrfnl merged commit 20ed3a6 into develop Jun 18, 2021
@jrfnl jrfnl deleted the add/editorconfig-file branch June 18, 2021 10:12
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.

2 participants