Hacktoberfest 2026:メンテナが10月に向けて印を付けた、オープンで初心者向けの issue。 Hacktoberfest の issue を見る

Allow asterisks in comments

オープン
#111 コメント 0 件 リアクション 3 件 担当者 0 名 GitHub で見る

まだ誰も着手していません。

評価

難易度
3/5
見積もり時間
1〜2日
初心者へのやさしさ
45/100
issue の種類
バグ
明瞭さ
おおむね明確
活発さ
停滞
技術スタック
javascript, nodejs
領域
tooling

調査の方向性

/* コメントを処理する parser のエントリポイントを特定し、/* Upload *.dSYM to Firebase */ で失敗を再現します。コメント本体がどのようにスキャンされるかを追跡し、そのうえで内部のアスタリスクが受け入れられる一方、閉じる */ によって SyntaxError なしでコメントが引き続き終了することを確認します。

索引モデルが issue の本文から書いたものです。

説明

bug

Once the parser passes a /* token, it proceeds to search for a */ token, throwing a SyntaxError as soon as it encounter an asterisk not immediately followed by a slash. Comments can contain asterisks though. Many of these comments in project files are autogenerated, for example based on a file name which could have an asterisk in it.

Could the parser ignore * characters between /* and */, continuing its search without issue until it finds the closing */?

To reproduce the below, call the parser on any (file containing a) line such as /* Upload *.dSYM to Firebase */:

Thrown:
{ [SyntaxError: Expected "*/" or [^*] but "*" found.
]
  message: 'Expected "*/" or [^*] but "*" found.',
  expected:
   [ { type: 'class',
       parts: [Array],
       inverted: true,
       ignoreCase: false },
     { type: 'literal', text: '*/', ignoreCase: false },
     { type: 'class',
       parts: [Array],
       inverted: true,
       ignoreCase: false },
     { type: 'literal', text: '*/', ignoreCase: false } ],
  found: '*',
  location:
   { start: { offset: 697, line: 13, column: 35 },
     end: { offset: 698, line: 13, column: 36 } },
  name: 'SyntaxError' }
主要言語
JavaScript
スター
217
フォーク
102
平均マージ
2日 9時間
マージ済み PR(30日)
8

コントリビューションガイド

コントリビューションガイドを開く

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

apache/cordova-node-xcode のほかの issue

apache/cordova-node-xcode の issue をすべて見る

似ている issue

JavaScript の issue をもっと見る

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。