-
Notifications
You must be signed in to change notification settings - Fork 1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
text-lintに関する記事 first commit #1673
base: main
Are you sure you want to change the base?
Conversation
✅ Deploy Preview for peaceful-bassi-80a807 ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
フィードバックとガイドラインフィードバック対象の記事「【補足記事】textlint-filter-rule-allowlistをkernelから使ってみよう」は、以下の点で優れています:
改善点:
ガイドライン以下は、PR作成者向けの具体的なガイドラインです:
以上の内容を参考に、PRの完成度を高めてください! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ヘッダーの日付について
記事のヘッダーに記載されている日付が 2025-01-27
となっています。本日は 2025-01-23
なので、未来の日付として問題ありません。ただし、公開予定日が1週間以上先の場合は意図的かどうか確認をお願いします!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
タイトルの最適化について
現在のタイトルは「【補足記事】textlint-filter-rule-allowlistをkernelから使ってみよう」となっています。このタイトルは内容を的確に表していますが、以下のようなバリエーションも検討してみてはいかがでしょうか?
- 「textlintのallowlistルールをKernelで活用する方法」
- 「Kernelからtextlintを使いこなす:allowlistルールの実践」
あくまで提案なので、最終的な判断はお任せします!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ファイル名の確認
ファイル名が 0127_textlint-tuning-allowlist.md
となっていますが、ヘッダーの日付 2025-01-27
に基づいて正しい形式です!素晴らしいです!
@shohei-yamashit この画像を記事に添付する場合は、マークダウンヘッダ部に
画像の利用は任意です。記事の内容に合っていない場合や画像を添付したくない場合は対応不要です。 画像を再度生成する場合は、このPRのコメントに以下の内容で投稿してください。
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
レビューしました。
}; | ||
} | ||
``` | ||
textlint-filter-rule-allowlistはあくまでJSのスクリプトなので、型情報はありません。 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
JSがJavaScriptの略なので「JSのスクリプト」は少し冗長な気がしました。
Publicな文章で拡張子以外でJSと略さない方がいいような気がします(好みの問題な気もするので対応しなくてもいいです)。
- 参照記事のリンクを追加 - moduleInterop の説明を「kernelによる制御」の章に追加 - JS→JavaScript - コードの変数の前後に"```"を追加 - 文中のKernel→kernel - その他冗長な表現の修正 #1673 (comment) #1673 (comment) #1673 (comment)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
よろしくお願いします