Unindent-chained-methods option. resolves #482

[aecepoglu]

unindent_chained_method = true will produce this:

fun1()
.fun2();

instead of

fun1()
    .fun2();

[bitwiseman]

Looks like the CI isn't updating PR statuses...

[aecepoglu]

@bitwiseman I guess I'll play with my PR a bit more then. By the time the CI is up and running I'll have the same open

[bitwiseman]

Okay, I've been swamped but i should have some time to look at it this week.

[brandondrew]

Excellent. Looking forward to this. Thanks for the contribution!

[aceysmith]

Bump?

[bitwiseman]

I don't see a new version of this pull request.

[aecepoglu]

I didn't forget about tthis. I'm just currently a bit occupied

On Wed, May 25, 2016 at 04:32:39PM -0700, Liam Newman wrote:

I don't see a new version of this pull request.

---

You are receiving this because you modified the open/close state.
Reply to this email directly or view it on GitHub:
#927 (comment)

[bitwiseman]

@aecepoglu - believe me I know how that goes. 😄 Whenever you find opportunity.

[bitwiseman]

@aecepoglu - any reason not to merge this as it was?

[aecepoglu]

It doesn't remove redundant empty lines

On Fri, Jun 17, 2016 at 11:18:07AM -0700, Liam Newman wrote:

@aecepoglu - any reason not to merge this as it was?

---

You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub:
#927 (comment)

[pietrovismara]

+1 please!!!

[w0rp]

I'll 👍 this one too. Is there any chance this could get picked up by someone? If it works and passes all of the tests, could we merge it?

[aecepoglu]

OK, I'm taking this back on. Give me a week or two and I'll have it up and running

On Sun, Sep 04, 2016 at 10:00:02AM -0700, w0rp wrote:

I'll 👍 this one too. Is there any chance this could get picked up by someone? If it works and passes all of the tests, could we merge it?

You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub:
#927 (comment)

[pietrovismara]

Almost 2 months later... :(

[w0rp]

It's funny, because in the intervening time I've actually started to up-indent my chains.

[aecepoglu]

Guys, I keep telling myself I'll finish this PR and be done with it but so many things have been popping up lately that I cannot get myself to do it.

[bitwiseman]

@aecepoglu - Welcome to open-source. 😄

[borwahs]

@aecepoglu - Do you have an example of the issue you described keeping this from being finished? I wouldn't mind helping out.

[aecepoglu]

@borwahs Okay my memory is a little cloudy but... The problem was that when I added my change, beautification didn't delete empty lines.

I wanted to change my approach rather than do it the way I have been doing.

[bitwiseman]

@borwahs @aecepoglu - Any progress on this? Just checking.

[borwahs]

@bitwiseman - my apologies! I got stuck in a project that consumed all my time (+ free time). I will try to pick this back up over the weekend.

[aiguofer]

This would be sweet! a friend I'm working with on a project hates me because js-beautify keeps indenting chained methods and he apparently feels strongly against that.... I don't really care either way, but I just want consistent code :)

[pietrovismara]

@borwahs Man i'm seriously considering to stop using a beautifier. Please please please.

[bitwiseman]

@pietrovismara - Any help you can offer is appreciated.

[dogramacigokhan]

Is there any update?

[aecepoglu]

OK, ok, @bitwiseman I'm going to solve this. I'll start in an hour and not work on anything else until I solve this.

[aecepoglu]

Re-opened at #1240