Make quoteProps: consistent the default value

[GauBen]

I just discovered that prettier supports wrapping and unwrapping object properties with quotes based on surrounding keys and I love it. It's so great for consistency, I feel like this should be the default value.

Examples:

.eslintrc.js

// Sorting in alphabetical order makes ' appear at the top
module.exports = {
  rules: {
    'arrow-body-style': 'error',
    'capitalized-comments': 'off',
    'max-depth': 'warn',
    'object-shorthand': 'warn',
    complexity: 'warn', // Ugh
    curly: ['warn', 'multi-or-nest', 'consistent'],
  },
};

// With quoteProps: consistent
module.exports = {
  rules: {
    'arrow-body-style': 'error',
    'capitalized-comments': 'off',
    'complexity': 'warn',
    'curly': ['warn', 'multi-or-nest', 'consistent'],
    'max-depth': 'warn',
    'object-shorthand': 'warn',
  },
};

HTTP headers

// Before
fetch('/post', {
  method: 'POST',
  headers: {
    'Content-Type': 'application/json',
    Authorization: `Basic ${token}`, // Does not look very idiomatic
  },
  body: '{}',
});

// After
fetch('/post', {
  method: 'POST',
  headers: {
    'Authorization': `Basic ${token}`, // Much better
    'Content-Type': 'application/json',
  },
  body: '{}',
});

I'm curious to see what the community thinks about it

[kachkaev]

👋 @GauBen! This question was brought up in #6931. We decided to not use consistent based on the size of git diffs.

[GauBen]

Thanks for the heads-up, I was sure someone brought up the idea earlier but I couldn't find it