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

[MSHARED-1408] Refresh IntelliJ formater #14

Merged
merged 1 commit into from
Jun 8, 2024
Merged

Conversation

slawekjaranowski
Copy link
Member

Exported by IntelliJ IDEA 2024.1.2 (Ultimate Edition)

Exported by IntelliJ IDEA 2024.1.2 (Ultimate Edition)
@@ -123,7 +131,7 @@ under the License.
<codeStyleSettings language="XML">
<indentOptions>
<option name="INDENT_SIZE" value="2" />
<option name="CONTINUATION_INDENT_SIZE" value="4" />
<option name="CONTINUATION_INDENT_SIZE" value="2" />
Copy link
Contributor

Choose a reason for hiding this comment

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

Is. that the value used by the formatter ?

Copy link
Member Author

Choose a reason for hiding this comment

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

simply I changed my local config to format in the same way as spotles does
this is result of my export - so probably is used 😄

Copy link
Member Author

Choose a reason for hiding this comment

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

any other comments?

Copy link
Member Author

Choose a reason for hiding this comment

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

ok, as it is for developer as help only I will merge.
In case of any feedback we will simply update it.

@slawekjaranowski slawekjaranowski requested a review from gnodet June 3, 2024 18:46
@slawekjaranowski slawekjaranowski merged commit ec66b7d into master Jun 8, 2024
31 checks passed
@slawekjaranowski slawekjaranowski deleted the MSHARED-1408 branch June 8, 2024 09:30
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