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

Issue 21 エラーメッセージの改善 #25

Merged
merged 4 commits into from
Apr 24, 2020

Conversation

KEINOS
Copy link
Contributor

@KEINOS KEINOS commented Apr 22, 2020

Issue #21 対応開始の意思表示のため Draft PR をあげました。

当方の開発環境に Node.js をローカルで入れられないため Dockerfile を暫定的に追加しましたが、Draft を解除(レビュー&マージ依頼)する際に削除して PR いたします。

Node.js 初心者&初 node.js の PR です。途中、気になる点や問題ありましたらご指摘いただければ嬉しいです。

エラー時何調の文体を期待しているか明確にした。
「〜が〜調」→「〜調の文体に、次の〜調の箇所があります: 〜」
textlint-ja#21
Clean up temporary Dockerfile for non Node.js env
@KEINOS KEINOS marked this pull request as ready for review April 22, 2020 05:02
@KEINOS
Copy link
Contributor Author

KEINOS commented Apr 22, 2020

@azu さま

エラー発生(要修正箇所)の行番号・列番号の表示はレベルが高すぎて実装できませんでした。
取り急ぎ、メッセージ内容の改善提案だけ PR いたします。

おてすきにレビューをお願いいたします。

Copy link
Member

@azu azu left a comment

Choose a reason for hiding this comment

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

良さそうな気がします。

@azu azu merged commit a310688 into textlint-ja:master Apr 24, 2020
@azu
Copy link
Member

azu commented Apr 24, 2020

PRありがとうございます!
https://github.com/textlint-ja/textlint-rule-no-mix-dearu-desumasu/releases/tag/4.0.1 としてリリースしました

@KEINOS KEINOS deleted the Issue-21_improve-output-msg branch April 24, 2020 23:30
@KEINOS
Copy link
Contributor Author

KEINOS commented Apr 24, 2020

@azu

マージありがとうございます!やった!

@KEINOS
Copy link
Contributor Author

KEINOS commented Apr 24, 2020

おっとー、README.md がメッセージ内容の変更に追随してませんでした。別途 PR いたします。

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.

None yet

2 participants