Conflict between object-shorthand-properties-first and object-literal-sort-key

[brunolemos]

Bug Report

• TSLint version: ^5.8.0
• TypeScript version: ^2.6.2
• Running TSLint via: VSCode

TypeScript code being linted

const a = 'a'
const c = 'c'
const obj = {
  a,
  b: 'b',
  c,
}

with tslint.json configuration:

{
  "extends": ["tslint:recommended", "tslint-config-airbnb", "tslint-react"]
}

Actual behavior

The rules conflict. There isn't a single field ordering that matches both rules, resulting on a non-fixable warning.

a, b, c

a, c, b

Expected behavior

Not conflict.

PS: I see this was mentioned on #2040 by @ajafff

Also I noticed object-shorthand-properties-first is from another repo but the fix makes more sense to be inside object-literal-sort-key

[brunolemos]

I was about to make a PR but it seems there is already #2831 that fixes this

[brunolemos]

I created a PR anyway (#3607) because the existing one was outdated and included other features