Atom-beautify: Unnecessary chain method indentation

[qwelias]

Description

The results of beautification are not what I expect.

Expected Results

expected:

Steps to Reproduce

input:

new sql.Request().query(query.query).then((docs) => {
    res.json({
                result: true
        });
}).catch((e) => {
    e.status = e.status || 500;
    Log(e.stack || e);
    res.status(e.status).end();
});

Beautify

output:

Debug

debug.md

[prettydiff]

It appears the output and expected output are identical aside from the extra indentation. This would require a new option from both Pretty Diff and JS Beautify to customize this behavior differently from the default.

I recommend opening new issues with each of those projects:

• https://github.com/beautify-web/js-beautify/issues
• https://github.com/prettydiff/prettydiff/issues

[Glavin001]

Closing this as third-party issue of Pretty Diff and/or JS-Beautify. Please let me know if a new option is needed in Atom-Beautify to support this.

[pietrovismara]

@Glavin001 After a long time, the enhancement has been implemented and merged by js-beautify (https://github.com/beautify-web/js-beautify/pull/1240). Any step that should be taken in order to have the latest js-beautify version in atom-beautify?

[bitwiseman]

@pietrovismara - The change is not released yet. It will be in 1.7.0.

[pietrovismara]

@bitwiseman I gave a look at 1.7.0 milestone.

Does this mean that it will take several months at least before having it released??

[bitwiseman]

@pietrovismara Coming this weekend.

[bitwiseman]

@pietrovismara - Done.

[pietrovismara]

@bitwiseman. Thank you so much. Guess it's only a matter of time til i get the update in Atom.