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/eslint #42

Merged
merged 9 commits into from
Dec 17, 2022
Merged

Fix/eslint #42

merged 9 commits into from
Dec 17, 2022

Conversation

HosokawaR
Copy link
Member

@HosokawaR HosokawaR commented Nov 25, 2022

背景

Resolves #24
その他の Warn / Error が残っている

解決策

細かいので各コミットメッセージを参照

Merge について

#51 が Merge されてから本 PR を Merge する

@HosokawaR
Copy link
Member Author

HosokawaR commented Nov 25, 2022

@azarashi2931

このブランチの yarn lint を手元で試すと以下のように Failed するのだけど、CI の yarn lint は Passed するのはなんでなんだろう?

$ yarn lint
yarn run v1.22.19
$ eslint . --ext .tsx,.ts --fix
Warning: React version not specified in eslint-plugin-react settings. See https://github.com/yannickcr/eslint-plugin-react#configuration .

/home/.../twinte/manaba-report-integration/src/ui/blacklistUtils.ts
  0:0  error  Parsing error: Cannot read properties of undefined (reading 'map')

✖ 1 problem (1 error, 0 warnings)

error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
$ echo $?
1

@azrsh
Copy link
Contributor

azrsh commented Dec 10, 2022

Reviewdog のフィルタの影響で、その PR で変更された行以外にエラーがあってもエラーが出ない
https://github.com/reviewdog/reviewdog#added-default

master ブランチの CI ではこれを無効化した方がいいかもね

@HosokawaR
Copy link
Member Author

HosokawaR commented Dec 12, 2022

なるほどー
最初から lint error があるコードが悪いので、仕方がない

master ブランチの CI ではこれを無効化した方がいいかもね

たしかに、賛成です。

@azrsh
Copy link
Contributor

azrsh commented Dec 14, 2022

これは別の PR で直すんでとりあえず今あるの lint error は直しちゃってほしい

@HosokawaR HosokawaR marked this pull request as ready for review December 15, 2022 19:40
@HosokawaR HosokawaR requested a review from azrsh December 15, 2022 19:40
@HosokawaR
Copy link
Member Author

"explicit-module-boundary-types" rule may be unnesesarry. って、コミットメッセージの一行目が感想なのおもろいな()

@HosokawaR
Copy link
Member Author

これは別の PR で直すんでとりあえず今あるの lint error は直しちゃってほしい

待たせてすみません
パッケージのバージョン周りで詰まっていたので遅れました

.eslintrc.json Show resolved Hide resolved
src/background/background.ts Show resolved Hide resolved
@HosokawaR HosokawaR requested a review from azrsh December 17, 2022 05:13
; Conflicts:
;	src/popup/settingSection.tsx
;	yarn.lock
@HosokawaR HosokawaR merged commit 01aca0b into master Dec 17, 2022
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.

log レベルのデバッグ情報がコンソールに表示される
2 participants