Mention GitHub issues in TODO comments

Author: strager

test/test-lex.cpp

@@ -352,7 +352,7 @@ TEST_F(test_lex, fail_lex_legacy_octal_numbers) {
       });
 }
 
-// TODO (👮🏾‍♀️) (when strict mode implemented) legacy octal number
+// TODO (#73) (when strict mode implemented) legacy octal number
 // literal tests to fail in strict mode
 
 TEST_F(test_lex, legacy_octal_numbers_cannot_contain_underscores) {
@@ -1050,7 +1050,7 @@ TEST_F(test_lex, lex_regular_expression_literals) {
 
   // TODO(#187): Report invalid escape sequences.
 
-  // TODO(strager): Report invalid characters and mismatched brackets.
+  // TODO(#203): Report invalid characters and mismatched brackets.
 }
 
 TEST_F(test_lex, lex_regular_expression_literal_with_digit_flag) {

test/test-lint.cpp

@@ -2536,7 +2536,7 @@ TEST(test_lint_magic_arguments, catch_variable_shadows_magic_arguments) {
   EXPECT_THAT(v.errors, IsEmpty());
 }
 
-// TODO(strager): 'arguments' should not be declared in arrow functions.
+// TODO(#204): 'arguments' should not be declared in arrow functions.
 
 TEST(test_lint_typeof, using_undeclared_variable_in_typeof_is_not_an_error) {
   const char8 use[] = u8"v";

test/test-lsp-error-reporter.cpp

@@ -85,7 +85,7 @@ TEST_F(test_lsp_error_reporter, assignment_before_variable_declaration) {
             "variable assigned before its declaration");
   EXPECT_EQ(diagnostics[0]["code"], "E001");
   EXPECT_EQ(diagnostics[0]["source"], "quick-lint-js");
-  // TODO(strager): Show the declaration as relatedInformation.
+  // TODO(#200): Show the declaration as relatedInformation.
 }
 
 TEST_F(test_lsp_error_reporter, assignment_to_undeclared_variable) {

test/test-options.cpp

@@ -230,9 +230,9 @@ TEST(test_options, invalid_vim_file_bufnr) {
   }
 }
 
-// TODO(strager): Report warning for trailing (ununsed) --vim-file-bufnr.
+// TODO(#201): Report warning for trailing (ununsed) --vim-file-bufnr.
 
-// TODO(strager): Report warning for using --vim-file-bufnr without
+// TODO(#201): Report warning for using --vim-file-bufnr without
 // --output-format.
 
 TEST(test_options, invalid_option) {

test/test-parse-var.cpp

@@ -779,7 +779,7 @@ TEST(test_parse, new_style_variables_cannot_be_named_let) {
     EXPECT_EQ(v.variable_declarations[0].kind, variable_kind::_import);
   }
 
-  // TODO(strager): export implies strict mode (because modules imply strict
+  // TODO(#73): export implies strict mode (because modules imply strict
   // mode).
   if ((false)) {
     spy_visitor v;