one-var with --fix is adding multiple empty lines between var declarations on keeping on the same line

Tell us about your environment

• ESLint Version: v5.2.0

• Node Version: 8.11.3

• npm Version: 5.6.0

What parser (default, Babel-ESLint, etc.) are you using? default

Please show your full configuration:

Configuration

{
  "env": {
    "browser": true
  },
  "extends": "eslint:recommended",
  "parserOptions": {
    "ecmaVersion": 5
  },
  "rules": {
    "indent": ["error", 2],
    "linebreak-style": ["error", "unix"],
    "quotes": ["error", "single"],
    "semi": ["error", "always"],
    "one-var": ["error", "never"]
  }
}

What did you do? Please include the actual source code causing the issue, as well as the command that you used to run ESLint.

Input A

var a = 1, b = 2, c = 3;

Output A

var a = 1; var b = 2; var c = 3;

Input B

var a = 1, 
  b = 2, 
  c = 3;

Output B

var a = 1;
 
    
var b = 2;
 
    
var c = 3;

eslint --quiet --fix src/

What did you expect to happen? Do not include multiple lines between variables declaration

Expected output on both cases

var a = 1; 
var b = 2;
var c = 3;

[carlos-algms]

This does feel like a bug rather than the intended behaviour. --fix shouldn’t do more than what the rule intends it to do

It is a bug when the fix cause more problems, like including useless empty lines between declarations. That is not acceptable on any style-guide.

[podkot]

reopen please, the current behavior is quite annoying when --fix is applied on a large legacy codebase