Pad nested callbacks with newlines

Please describe what the rule should do: Pad large callbacks with new lines.

What new ECMAScript feature does this rule relate to? Not applicable - general formatting.

What category of rule is this? (place an “X” next to just one item) [ ] Warns about a potential error (problem) [x] Suggests an alternate way of doing something (suggestion) [ ] Other (please specify:)

Provide 2-3 code examples that this rule will warn about:

const assert = require('assert');
const pg = require('pg');
const DBManager = require('../../src/database/DBManager.js');

describe('DBManager', () => {
// pad with new line here
    describe('#get', () => {
// pad with new line here
        it('should return an instance of pg.Pool', done => {
            assert.equal(true, DBManager.get() instanceof pg.Pool);
            done();
        });
// pad with new line here
    });
// pad with new line here
});

Why should this rule be included in ESLint (instead of a plugin)? It helps break up large function callback nesting as seen in mocha tests.

Are you willing to submit a pull request to implement this rule? I’m juggling a handful of projects currently, so unfortunately no.

[mdjermanovic]

@brenr thanks for the rule proposal!

Given the changes to our rules policy, I’d suggest making a custom rule, which could be based on the padded-blocks rule.

[eslint-deprecated[bot]]

Unfortunately, it looks like there wasn’t enough interest from the team or community to implement this change. While we wish we’d be able to accommodate everyone’s requests, we do need to prioritize. We’ve found that issues failing to reach accepted status after 21 days tend to never be accepted, and as such, we close those issues. This doesn’t mean the idea isn’t interesting or useful, just that it’s not something the team can commit to.

Thanks for contributing to ESLint and we appreciate your understanding.