Eslint-plugin-react: jsx-pascal-case warns on valid casing when using non-English characters

[bjornreppen]

This rule only works when strictly using the 26 english characters hard coded in the regex.

const PASCAL_CASE_REGEX = /^([A-Z0-9]|[A-Z0-9]+[a-z0-9]+(?:[A-Z0-9]+[a-z0-9]))$/;

In order to support all cased languages, how do you feel about modifying the casing check for example using a technique similar to this (to detect letters):

function isLetter(c) {
  return c.toLowerCase() != c.toUpperCase();
}

[ljharb]

I'm not sure about that specific technique, but I'm very open to adding test cases that fail with the current method, and then fixing the implementation to pass those tests.

[bjornreppen]

Example of failing test case:

{ code: '<Surströmming />' }

[cquezel]

I don't know the implementation details, but this might test a case not covered by the original proposed test.

{ code: '<Éurströmming />' }

[LarsAndreasNav]

Any update on this? Our component "Flervalgsspørsmål" fails this rule.

[InExtremaRes]

Same here, a component called Año (year in spanish) fails.

[ljharb]

No update, because nobody's sent a PR.

[Svish]

Added a PR (#2557) now to support Unicode characters 🙂👍