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

adding an editorconfig that seems to match the project style #20

Merged
merged 1 commit into from
Oct 25, 2017

Conversation

SignpostMarv
Copy link
Contributor

insert_final_newline = false seems to be ignored by the editorconfig plugin for Atom, but other than that this seems to match what's on master.

indent_size = 2

[composer.json]
indent_style = tab
Copy link
Member

Choose a reason for hiding this comment

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

what if we change composer indent by spaces?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I think I did that on the static-analysis branch.
I could change it to spaces on this branch if you'd prefer ?

@goetas goetas merged commit a5882ed into goetas-webservices:master Oct 25, 2017
@goetas
Copy link
Member

goetas commented Oct 25, 2017

I like it anyway.
feel free to change it to space on the other branch

@SignpostMarv SignpostMarv deleted the editorconfig branch October 25, 2017 18:13
@SignpostMarv SignpostMarv mentioned this pull request Oct 25, 2017
3 tasks
SignpostMarv added a commit to SignpostMarv/xsd-reader that referenced this pull request Oct 27, 2017
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.

None yet

2 participants