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

Feat/consistent indent new chain method #456

Conversation

clementdessoude
Copy link
Contributor

What changed with this PR:

Example

// Input
public void method() {
    new Foo(stuff, thing, "auaaaaaaaaa some very long stuff", "some more").bar(10);
    foo(stuff, thing, "some very longuuuuuuuuuuuuuu stuff", "some more").bar(10);
}

// Before PR
public void method() {
    new Foo(stuff, thing, "auaaaaaaaaa some very long stuff", "some more")
    .bar(10);
    foo(stuff, thing, "some very longuuuuuuuuuuuuuu stuff", "some more")
        .bar(10);
}

// Output
public void method() {
    new Foo(stuff, thing, "auaaaaaaaaa some very long stuff", "some more")
      .bar(10);
    foo(stuff, thing, "some very longuuuuuuuuuuuuuu stuff", "some more")
      .bar(10);
}

Relative issues or prs:

Fix #448

@clementdessoude clementdessoude force-pushed the feat/consistent-indent-new-chain-method branch from 315521a to f1f39b3 Compare March 13, 2021 17:46
@clementdessoude clementdessoude force-pushed the feat/consistent-indent-new-chain-method branch from f1f39b3 to d06ff06 Compare March 13, 2021 17:52
@jhaber
Copy link
Contributor

jhaber commented Mar 13, 2021

👍 thanks for the fix!

@clementdessoude clementdessoude merged commit 69a44cf into jhipster:main Mar 13, 2021
@jhaber
Copy link
Contributor

jhaber commented Mar 13, 2021

If you push a new release I can update the maven plugin as well

@clementdessoude
Copy link
Contributor Author

@jhaber I created a PR here: HubSpot/prettier-maven-plugin#38 :)

This was referenced Mar 15, 2021
This was referenced Mar 15, 2021
@clementdessoude clementdessoude deleted the feat/consistent-indent-new-chain-method branch June 18, 2021 19:14
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.

Inconsistent indentation between chained method call on method and on new object
2 participants