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 #1437

Merged
merged 2 commits into from
Sep 1, 2021
Merged

Add .editorconfig file #1437

merged 2 commits into from
Sep 1, 2021

Conversation

SerhiiNahornyi
Copy link
Collaborator

No description provided.

@SerhiiNahornyi SerhiiNahornyi changed the title Add/editor/config Add .editorconfig file Aug 19, 2021
indent_size = 4
indent_style = space
insert_final_newline = true
max_line_length = 120
Copy link
Contributor

Choose a reason for hiding this comment

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

Will it affect JSON files?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

No

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

We have specific json properties, which gives json lines possibility to be as long as needed

ij_java_wrap_first_method_in_call_chain = false
ij_java_wrap_long_lines = false

[.editorconfig]
Copy link
Contributor

Choose a reason for hiding this comment

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

editorconfig for editorconfig? oO

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Yes, seems this is useful config

@rpanchyk rpanchyk merged commit a2f53fe into master Sep 1, 2021
@rpanchyk rpanchyk deleted the add/editor/config branch September 1, 2021 10:51
javaAndScriptDeveloper pushed a commit that referenced this pull request Sep 14, 2021
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