Skip to content
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

textlint の導入 #238

Merged
merged 3 commits into from
Jan 13, 2025
Merged

textlint の導入 #238

merged 3 commits into from
Jan 13, 2025

Conversation

takaram
Copy link
Contributor

@takaram takaram commented Dec 31, 2024

textlintを導入し、CIで実行します。

Closes #71

モチベーション

約2年前に #69 でtextlintの導入が提案され「メンテナが増えたら検討」となっていました。
ここ数ヶ月でメンテナ・PR作成者も増えており、実際に表記揺れも発生している(#223 など)現状を鑑みて、textlintの必要性が高まっていると感じます。

実装方針

buildディレクトリ以下にpackage.jsonや設定ファイルを配置しています。

#71 の"活用例"のブログでも紹介されている textlint-rule-preset-japanese などの導入も検討しましたが、一番の目的は表記揺れ対策のため、今回は textlint-rule-prh のみを導入しています。
build/prh.ymlは、README_Glossary.mdに記載の用語の中から、textlint-rule-prh で対応できそうなもののみ設定しています。

textlintは、デフォルトではXML文書に非対応です。XML (DocBook) に対応させるにはプラグインの自作が必要そうで、現実的でないため、PhDでHTMLに変換したものを対象に実行しています。
これの実現のため、CIではXHTML形式のビルド成果物をアーティファクトに保存しています。結果的に #193 (comment) の提案と同じことをしています。

@takaram takaram force-pushed the textlint branch 2 times, most recently from d7f73d2 to e3dfcf9 Compare December 31, 2024 07:50
Copy link
Collaborator

@KentarouTakeda KentarouTakeda left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@takaram
ありがとうございます!マージさせて頂きます!!

@KentarouTakeda KentarouTakeda merged commit 1f20a94 into php:master Jan 13, 2025
2 checks passed
@takaram takaram deleted the textlint branch January 13, 2025 22:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

textlint の導入検討
2 participants