Skip to content
This repository was archived by the owner on Jan 20, 2025. It is now read-only.

Conversation

@uetchy
Copy link
Contributor

@uetchy uetchy commented Jan 13, 2020

Add textlint to lint documents.
fixes #2

@@ -1,5 +0,0 @@
module.exports = {
Copy link
Contributor

Choose a reason for hiding this comment

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

huskyはそのまま残していても良いと思いますがどうでしょう?

Copy link
Contributor Author

@uetchy uetchy Jan 13, 2020

Choose a reason for hiding this comment

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

一覧性向上のためpackage.jsonに移動させました

Copy link
Contributor

Choose a reason for hiding this comment

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

なるほど!了解です:+1:

@uetchy uetchy requested a review from Naturalclar January 13, 2020 03:43
@Naturalclar
Copy link
Contributor

READMEの和訳をMergeしたのでConflictが発生してる様です:bow:

Copy link
Contributor

@hirotaka hirotaka left a comment

Choose a reason for hiding this comment

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

このあたりあまりよくわかっていないのですが。。。。
LGTM 👍

@uetchy
Copy link
Contributor Author

uetchy commented Jan 13, 2020

運用してみて合わない部分があれば適時修正PR出していきましょう👍

@Naturalclar
Copy link
Contributor

@hirotaka
textlintは簡単に言えば自動文章構成ツールです。
「です」「ます」調を揃えたり、単語の記述を揃えたりと、文章の統一感を出すために使います。
これが入ったら、だれかがPull Requestを出したときにBotが自動的そこを見て指摘してくれるようになるので、今後のメンテナンスが楽になります:+1:

Copy link
Contributor

@Naturalclar Naturalclar left a comment

Choose a reason for hiding this comment

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

LGTM

@Naturalclar Naturalclar merged commit 6971044 into gatsbyjs:master Jan 13, 2020
@uetchy uetchy deleted the textlint branch January 13, 2020 03:59
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add Textlint

3 participants