Make @typescript-eslint/naming-convention ignore non alphanumeric keys

[GauBen]

Please see #53 for context

\W means [^a-zA-Z0-9_], making keys containing symbols (and thus quoted) ignored by the rule.

Fixes #53

[sindresorhus]

$ is a valid character in a JS identifier.

I guess you could borrow the regex from here: https://github.com/yeoman/stringify-object/blob/e6ef8e2aa276f79b86a04398df273d45e838a254/index.js#L111

[GauBen]

That's true, but I don't know how to capitalize "$lib", maybe it would make more sense to restrict the regex to "sentence-like" keys ([^a-zA-Z_]), without numbers and symbols.

[GauBen]

I'm back with a crazier edge-case:

export default class Ping {
  @Get()
  ''(): string {
//^^ Class Method name `` must match...
    return 'pong'
  }
}

[sindresorhus]

Actually, I found: https://typescript-eslint.io/rules/naming-convention/#ignore-properties-that-require-quotes

[GauBen]

Disclaimer: not tested

[sindresorhus]

Suggested change

	modifiers: ["requiresQuotes"]
	modifiers: [
	'requiresQuotes'
	]

[sindresorhus]

It needs to be tested for it to be merged.

[gutenye]

eslint-config-xo-typescript/index.js

Lines 63 to 73 in 286a88e

	// Allow these in non-camel-case when quoted.
	{
	selector: [
	'classProperty',
	'objectLiteralProperty'
	],
	format: null,
	modifiers: [
	'requiresQuotes'
	]
	}

Should be removed