Rule proposal: no-empty-line-separating-import

[gajus]

This is stylistic rule. I often see import statements separated one from another using arbitrary rules, e.g.

import yargs from 'yargs';
import getStdin from 'get-stdin';


import {
    lintFiles,
    lintText
} from './../../utilities';

import {
    failHandler,
    getTargetPaths,
    outputReport
} from './../../utilities/bin';

This rule would enforce no spacing between import statements, i.e.

import yargs from 'yargs';
import getStdin from 'get-stdin';
import {
    lintFiles,
    lintText
} from './../../utilities';
import {
    failHandler,
    getTargetPaths,
    outputReport
} from './../../utilities/bin';

[benmosher]

Just a heads up: if you want to implement + PR this and the others you've proposed, I'm more than willing to accept them.

I just don't have a lot of time ATM and the style-guide rules are lower priority than getting ready for ESLint 2, infrastructure, etc. so I don't expect to work on them any time soon.

[alexander-akait]

@benmosher maybe rename max-empty-line-separating-import with option number?

[jfmengels]

You can configure the order rule to do the wanted behavior. It's a but of a hack if you only want to enforce this trait, but that would be the way to go IMO. What do you think @benmosher?

[benmosher]

I'm cool either way. Seems reasonable to assume that intent to use a rule like this implies desire to use order.

[ljharb]

That assumption is incorrect - I do not want ordering but do want imports grouped with no newlines.

[caub]

I agree with ↑, https://github.com/benmosher/eslint-plugin-import/blob/master/docs/rules/order.md#newlines-between-ignorealwaysalways-and-inside-groupsnever lets ignore the newlines-between option, but not the group one

oh.. right, after reading more:

'import/order': ['error', {
      'groups': [['index', 'sibling', 'parent', 'internal', 'external', 'builtin']],
      'newlines-between': 'never'
    }],

works to allow mangling of all groups

[Devansu-Yadav]

@benmosher @ljharb Is this issue still open to take up? I don't see any PR created for this rule, so I was hoping to take a shot at implementing this if it's something useful to have 😅

For starters, I'm thinking of going through the code for newlines-between option (especially the never value that can be passed) of the order rule to understand how to avoid empty lines between imports. I'm also looking to play around with the AST Explorer to get an idea about the implementation.

Though, I'll need some input from your end to get some clarity on the implementation as well as what options we could configure for this rule 😅

[ljharb]

@Devansu-Yadav yup! no PR yet. altho let's make sure we're on the same page about implementation first.

[Devansu-Yadav]

@Devansu-Yadav yup! no PR yet. altho let's make sure we're on the same page about implementation first.

Sure, absolutely!

[Devansu-Yadav]

@Devansu-Yadav yup! no PR yet. altho let's make sure we're on the same page about implementation first.

@ljharb Any updates on how I should be proceeding with the implementation?

[ljharb]

What schema did you have in mind?