There's a problem with 'always' and TypeScript.

That rule is enabled in standard-with-typescript. And it should remain that way, for better type checking. So, the typescript config could possibly set object-shorthand back to 'properties' if it is ever upgraded to 'always'.

@mightyiam I believe that that lint only applies to interfaces, which is where the stricter typings comes into play. This rule would only enforce it on object literals, so I think it should be fine. I could totally be missing something though 😄

Also, I just realised that { a: () => {} } and { a () {} } has totally different semantics regarding this 🤔 Might be better to keep this as just properties if it forces on of those 🤔

I've gone through and merged the PRs on the repos that I have access to!

@LinusU Should we release this as a warn in 17.0.0? To prepare people for us making it an error in 18.0.0?

@voxpelli sounds very good 👍

Rebased and switched to warn ✅

Not compatible:

• subleveldown - Use object shorthand for properties Level/subleveldown#91
• fastparallel
• reusify - Use object shorthand for properties mcollina/reusify#11
• levelup
• memdown
• leveldown

Not yet upgraded:

• david - Use object shorthand for properties alanshaw/david#167
• bs58check - Use object shorthand for properties bitcoinjs/bs58check#25
• markdown-pdf - Use object shorthand for properties alanshaw/markdown-pdf#186
• bittorrent-tracker - style: use object shorthand for properties webtorrent/bittorrent-tracker#400
• simple-bin-help - Use object shorthand for properties bahmutov/simple-bin-help#84
• rocha - Use object shorthand for properties bahmutov/rocha#140
• subcommand - Use object shorthand for properties max-mapper/subcommand#9
• coininfo - Use object shorthand for properties cryptocoinjs/coininfo#107
• bip38
• base-input
• sodium-signatures
• glslify-bundle - Use object shorthand for properties glslify/glslify-bundle#9
• int-encoder
• geohub
• evp_bytestokey
• gl-blend-demo
• stackman - Use object shorthand for properties watson/stackman#11
• studynotes.org
• webtorrent.io
• instant.io
• karma-chrome-launcher - refactor: use object shorthand for properties karma-runner/karma-chrome-launcher#222
• nyc
• speakeasyjs.com
• multer - Switch to stream based API expressjs/multer#399
• front-matter
• extract-zip
• assets-webpack-plugin
• fastify
• klaw

🚀