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

ciでtextlintを動かす #160

Merged
merged 3 commits into from
Jun 4, 2024
Merged

ciでtextlintを動かす #160

merged 3 commits into from
Jun 4, 2024

Conversation

kyu08
Copy link
Owner

@kyu08 kyu08 commented Jun 2, 2024

close #158

  • lintエラーを修正する(npx textlint --fix .でいけそう)
  • 好みじゃないルールは無効化する
    • 全角のを使うように強制するルール(これ気のせいかも)

@@ -64,7 +64,7 @@

6月はたくさん本を読みました。毎日作業を始める前に1時間本を読むスタイルが自分にあっている感じでした。

Ruby の OOP 本は
RubyのOOP本は
Copy link

Choose a reason for hiding this comment

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

🚫 [textlint] <eslint.rules.ja-technical-writing/ja-no-mixed-period> reported by reviewdog 🐶
文末が"。"で終わっていません。 (ja-technical-writing/ja-no-mixed-period)


Evans本を読みました。成瀬さんのDDD本を読んでから望んだので前半はわりと理解できましたが後半はサッパリでした...。🤔

IntelliJ ちょっと重いなあ...ということで NeoVim に入門してみましたが Plugin がなかなかうまく動かなかったりして IntelliJ でやってることができる状態にするのめっちゃ大変やん...となり挫折しました。
IntelliJちょっと重いなあ...ということでNeoVimに入門してみましたがPluginがなかなかうまく動かなかったりしてIntelliJでやってることができる状態にするのめっちゃ大変やん...となり挫折しました。
Copy link

Choose a reason for hiding this comment

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

🚫 [textlint] <eslint.rules.ja-technical-writing/sentence-length> reported by reviewdog 🐶
Line 124 sentence length(110) exceeds the maximum sentence length of 100.
Over 10 characters. (ja-technical-writing/sentence-length)


応用情報受かりました!🎉

午後が結構不安だったのでめちゃめちゃ嬉しかったです。(午前78%, 午後62.5%だった)

わりと広く浅く学べたのでこれからがっつり学ぶ必要が出てきたときの土台となってくれれば...と期待しています。。

Next.js に入門しました。今まで component が増えてくると辛みが増していましたが component を`/pages`と`/component`に配置するやり方が読みやすさ的にも書きやすさ的にもいいなと感じました。
Next.jsに入門しました。今までcomponentが増えてくると辛みが増していましたがcomponentを`/pages`と`/component`に配置するやり方が読みやすさ的にも書きやすさ的にもいいなと感じました。
Copy link

Choose a reason for hiding this comment

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

🚫 [textlint] <eslint.rules.ja-technical-writing/no-doubled-joshi> reported by reviewdog 🐶
一文に二回以上利用されている助詞 "にも" がみつかりました。

次の助詞が連続しているため、文を読みにくくしています。

  • "にも"
  • "にも"

同じ助詞を連続して利用しない、文の中で順番を入れ替える、文を分割するなどを検討してください。
(ja-technical-writing/no-doubled-joshi)

@@ -108,9 +108,9 @@
round == initRound5
```

仕様上 `Points` の長さは 4 もしくは 5 になるのですが、elm-static-array を使って書くと `isDefaultRound`関数 のパターンマッチの中では仕様上ありえる値の分岐だけを扱えばいいことがわかると思います
仕様上 `Points` の長さは4もしくは5になるのですが、elm-static-arrayを使って書くと `isDefaultRound`関数のパターンマッチの中では仕様上ありえる値の分岐だけを扱えばいいことがわかると思います
Copy link

Choose a reason for hiding this comment

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

🚫 [textlint] <eslint.rules.ja-technical-writing/ja-no-weak-phrase> reported by reviewdog 🐶
弱い表現: "思います" が使われています。 (ja-technical-writing/ja-no-weak-phrase)

@@ -108,9 +108,9 @@
round == initRound5
```

仕様上 `Points` の長さは 4 もしくは 5 になるのですが、elm-static-array を使って書くと `isDefaultRound`関数 のパターンマッチの中では仕様上ありえる値の分岐だけを扱えばいいことがわかると思います
仕様上 `Points` の長さは4もしくは5になるのですが、elm-static-arrayを使って書くと `isDefaultRound`関数のパターンマッチの中では仕様上ありえる値の分岐だけを扱えばいいことがわかると思います
Copy link

Choose a reason for hiding this comment

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

🚫 [textlint] <eslint.rules.ja-technical-writing/sentence-length> reported by reviewdog 🐶
Line 111 sentence length(112) exceeds the maximum sentence length of 100.
Over 12 characters. (ja-technical-writing/sentence-length)

tags = ["開発環境", "ターミナル", "zsh", "cli"]
keywords = ["", ""]
tags =["開発環境", "ターミナル", "zsh", "cli"]
keywords =["", ""]
Copy link

Choose a reason for hiding this comment

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

🚫 [textlint] <eslint.rules.ja-technical-writing/max-comma> reported by reviewdog 🐶
This sentence exceeds the maximum count of comma. Maximum is 3. (ja-technical-writing/max-comma)

@@ -240,7 +240,7 @@
などが可能です。

筆者はウィンドウ管理機能の中のウィンドウの切り替え機能を使っています。
sainnhe/tmux-fzf を使うようにしたことでウィンドウ切り替えを
sainnhe/tmux-fzfを使うようにしたことでウィンドウ切り替えを
Copy link

Choose a reason for hiding this comment

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

🚫 [textlint] <eslint.rules.ja-technical-writing/ja-no-mixed-period> reported by reviewdog 🐶
文末が"。"で終わっていません。 (ja-technical-writing/ja-no-mixed-period)

@@ -16,7 +16,7 @@
---

## 概要
McKinsey & Company出身の伊賀泰代さんがリーダーシップについて書いた『採用基準 地頭より論理的思考力より大切なもの』を読んだ。ちょうど「リーダーシップとは」みたいなことが気になっていたので読んでみた。
McKinsey & Company出身の伊賀泰代さんがリーダーシップについて書いた『採用基準地頭より論理的思考力より大切なもの』を読んだ。ちょうど「リーダーシップとは」みたいなことが気になっていたので読んでみた。
Copy link

Choose a reason for hiding this comment

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

🚫 [textlint] <eslint.rules.ja-technical-writing/no-doubled-joshi> reported by reviewdog 🐶
一文に二回以上利用されている助詞 "より" がみつかりました。

次の助詞が連続しているため、文を読みにくくしています。

  • "より"
  • "より"

同じ助詞を連続して利用しない、文の中で順番を入れ替える、文を分割するなどを検討してください。
(ja-technical-writing/no-doubled-joshi)

@@ -39,7 +39,7 @@

「全員がリーダーシップを持つ組織」と聞くと「全員自分の意見ばかり主張していては物事が前に進まないのでは...?」と考えてしまいがちだが、ここではリーダーシップを「チームの使命を達成するために、必要なことをやる人」と定義している。

たしかにそう聞くと多角的な視点からよりよい意思決定ができそうな気がしてくる。チーム・組織の全員がリーダーシップを体現することができれば指示待ちが起こらず、生産性が高まりそうだなと感じた。
たしかにそう聞くと多角的な視点からよりよい意思決定ができそうな気がしてくる。チーム・組織の全員がリーダーシップを体現できれば指示待ちが起こらず、生産性が高まりそうだなと感じた。
Copy link

Choose a reason for hiding this comment

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

🚫 [textlint] <eslint.rules.ja-technical-writing/no-doubled-joshi> reported by reviewdog 🐶
一文に二回以上利用されている助詞 "が" がみつかりました。

次の助詞が連続しているため、文を読みにくくしています。

  • "が"
  • "が"

同じ助詞を連続して利用しない、文の中で順番を入れ替える、文を分割するなどを検討してください。
(ja-technical-writing/no-doubled-joshi)

tags = ["開発環境", "ターミナル", "zsh", "cli", "Neovim", "Tmux"]
keywords = ["", ""]
tags =["開発環境", "ターミナル", "zsh", "cli", "Neovim", "Tmux"]
keywords =["", ""]
Copy link

Choose a reason for hiding this comment

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

🚫 [textlint] <eslint.rules.ja-technical-writing/max-comma> reported by reviewdog 🐶
This sentence exceeds the maximum count of comma. Maximum is 3. (ja-technical-writing/max-comma)

@kyu08 kyu08 marked this pull request as ready for review June 4, 2024 15:35
@kyu08 kyu08 merged commit afa940f into main Jun 4, 2024
2 of 3 checks passed
@kyu08 kyu08 deleted the textlint-ci branch June 4, 2024 15:36
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.

CIでtextlintを動かす
1 participant