svelte

Commit Graph

Author	SHA1	Message	Date
Rich Harris	cfdc8902ab	run prettier (spaces -> tabs)	7 years ago
asweingarten	02afdb03fa	[1083] Svelte should throw a compile time error when illegal characters are used in computed names Approach: For each property name, construct a string that defines a function and see if parsing that string with Acorn throws an exception. If it does, assemble an informative error message that states which property is invalid, the first invalid character, and the location of that character within the name. Changes to codebase: - Added new validator test "properties-computed-must-be-valid-function-names" - Added new check into src/validate/js/propValidators/computed.ts, "checkForValidIdentifiers" - this check was added to src/validate/js/utils/checkForValidIdentifiers.ts like the other checks in "computed.ts"	7 years ago

Author

SHA1

Message

Date

Rich Harris

cfdc8902ab

run prettier (spaces -> tabs)

asweingarten

02afdb03fa

[1083] Svelte should throw a compile time error when illegal characters are used in computed names

Approach:
  For each property name, construct a string that defines a function and see if parsing that string with Acorn throws an exception.
  If it does, assemble an informative error message that states which property is invalid, the first invalid character, and the location of that character within the name.

Changes to codebase:
- Added new validator test
"properties-computed-must-be-valid-function-names"
- Added new check into src/validate/js/propValidators/computed.ts,
"checkForValidIdentifiers"
  - this check was added to
  src/validate/js/utils/checkForValidIdentifiers.ts like the other
  checks in "computed.ts"

2 Commits (cfdc8902ab2f2f850832045a7266a48fdcbe3885)