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

fix text lint errors in reconciliation.md #116

Closed
wants to merge 1 commit into from
Closed

Conversation

koba04
Copy link
Member

@koba04 koba04 commented Feb 12, 2019

This is a PR to fix textlint errors introduced by #101, which are caused by new lint rules added by #98

@koba04
Copy link
Member Author

koba04 commented Feb 12, 2019

In order to avoid this, I've enabled "Require branches to be up to date before merging" option.

@koba04
Copy link
Member Author

koba04 commented Feb 12, 2019

closed by aed3392

@koba04 koba04 closed this Feb 12, 2019
@smikitky
Copy link
Member

@koba04 すみません黙って直していたら被りました…。こういう細かい誤字系は直コミットでも大丈夫ですよね?

"Require branches to be up to date before merging" はGitHubのUI上で大量の差分が出てきて「こりゃ使い物にならない」と思って一目見て止めた気がします。が、もしかしたら差分表示オプションをいじれば大丈夫かもしれません。

@koba04
Copy link
Member Author

koba04 commented Feb 12, 2019

@smikitky

"Require branches to be up to date before merging" はGitHubのUI上で大量の差分が出てきて「こりゃ使い物にならない」と思って一目見て止めた気がします。が、もしかしたら差分表示オプションをいじれば大丈夫かもしれません。

お、そうなんですね。これを有効にすると、多くの場合mergeの前に「Update branch」ボタンを押す必要が出てくるという手間が発生するくらいかなと思ったのですが、差分が出てくるというケースがちょっとわからないですね...。
このオプションが有効になっていないと今後も同じことが発生しそうだなとは感じました。

@smikitky
Copy link
Member

@koba04 多分2度目以降に "Files Changed" を見ると前に訪問した時からの差分が出るので、それ絡みにで一時的に大量のファイルの差分が出たのを勘違いしたのかもしれません…とりあえずこれで様子を見るので良いと思います

@koba04
Copy link
Member Author

koba04 commented Feb 12, 2019

@smikitky はい、有効にしてみて問題が出たら無効にしましょう。

@smikitky smikitky deleted the fix-textlint-error branch February 23, 2019 13:25
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.

2 participants