Merge pull request #201 from aspeddro/fix-unpack

allow `unpack` as a valid identifier

Author: nkrkv

grammar.js

@@ -66,6 +66,7 @@ module.exports = grammar({
     ],
     [$._jsx_attribute_value, $.pipe_expression],
     [$.function_type_parameters, $.function_type],
+    [$._reserved_identifier, $.module_unpack]
   ],
 
   conflicts: $ => [
@@ -1495,7 +1496,8 @@ module.exports = grammar({
     uncurry: $ => '.',
 
     _reserved_identifier: $ => choice(
-      'async'
+      'async',
+      'unpack'
     )
   },
 });

queries/highlights.scm

@@ -114,13 +114,14 @@
   "assert"
   "await"
   "with"
-  "unpack"
   "lazy"
   "constraint"
 ] @keyword
 
 ((function "async" @keyword))
 
+(module_unpack "unpack" @keyword)
+
 [
   "if"
   "else"

test/corpus/let_bindings.txt

@@ -8,6 +8,7 @@ let c = #foo
 let list = 1
 let a = list
 let async = 1
+let unpack = 2
 export d = 5
 
 ---
@@ -25,6 +26,8 @@ export d = 5
     (let_binding (value_identifier) (value_identifier)))
   (let_declaration
     (let_binding (value_identifier) (number)))
+  (let_declaration
+    (let_binding (value_identifier) (number)))
   (let_declaration
     (let_binding (value_identifier) (number))))