Merge pull request #241 from nafg/identifier-quotedifnecessary--cover…

…-more-cases Identifier.quotedIfNecessary: cover more cases
nafg · Dec 14, 2023 · 89c2ff6 · 89c2ff6
2 parents c6f6d51 + 2494650
commit 89c2ff6
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/project/Identifier.scala b/project/Identifier.scala
@@ -9,7 +9,11 @@ case class Identifier(value: String) {
 object Identifier {
   val keywords                            = Set("type", "true", "false")
   def quotedIfNecessary(value: String)    =
-    if (keywords.contains(value) || value.contains('-') || value.headOption.forall(_.isDigit))
+    if (
+      keywords.contains(value) ||
+      value.headOption.exists(!Character.isJavaIdentifierStart(_)) ||
+      value.exists(!Character.isJavaIdentifierPart(_))
+    )
       "`" + value + "`"
     else
       value