search

hiroyuki-ichikawa / fujicity_covid19

富士市版新型コロナ対策サイト
MIT License
1 stars 6 forks source link

Fix/issue #32 add year to date #34

Closed aktnk closed 3 years ago

aktnk commented 3 years ago

👏 解決する issue / Resolved Issues

📝 関連する issue / Related Issues

なし

⛏ 変更内容 / Details of Changes

📸 スクリーンショット / Screenshots

aktnk commented 3 years ago

eslint_error が発生している。 https://github.com/hiroyuki-ichikawa/fujicity_covid19/pull/34/checks?check_run_id=1845133341#step:4:31 ログを見ると、今回の修正ではなく、Github Actions側の設定にあるように見える・・・ しかし、明確な原因はわからない。

hrsano645 commented 3 years ago

ログの差分見てみましたが、GitHub Actions Ubuntuで利用するソフトウェアが更新されていて、nodeのバージョンがマイナーアップしているぐらいしか違いないですね。reviewdogの問題かな？

前回うまく行ってる例: https://github.com/hiroyuki-ichikawa/fujicity_covid19/runs/1797261881?check_suite_focus=true

aktnk commented 3 years ago

metroの設定をみてみると、2点の違いがある。 https://github.com/tokyo-metropolitan-gov/covid19/blob/b998955150c144b99e4e3540e94eb1348c3ccfb4/.github/workflows/reviewdog.yml#L27

hrsano645 commented 3 years ago

浜松web界隈で相談したら、issue/PRいただきました。先にmergeしておこうと思います。 Thanks @euledge 🎉 https://github.com/hiroyuki-ichikawa/fujicity_covid19/issues/35

aktnk commented 3 years ago
hrsano645 commented 3 years ago

@aktnk #36 のPRをdev版でmergeしたのでこのPRに取り込んで再度pushしなおしてみてください。（多分それでreviewdogのワークフローも更新されて動くはずです。

hrsano645 commented 3 years ago

metroの設定をみてみると、2点の違いがある。 https://github.com/tokyo-metropolitan-gov/covid19/blob/b998955150c144b99e4e3540e94eb1348c3ccfb4/.github/workflows/reviewdog.yml#L27

  • ubuntu:latestを使っている
  • eslint review の前にyarn install を実行している この辺を試してみるかですね。

この話今頃気が付きましたが（更新し忘れてました）、浜松web界隈の方々にも同じ話でPR頂いてました。東京都版は8月頃に導入されていたらしいです。

https://github.com/tokyo-metropolitan-gov/covid19/pull/5216

yarnかnpmかは、どちらでもよさそうです。メトロ版を追いかけて同様に依存関係の解消が出来そうなときにはワークフローもメトロ側に合わせるでいいかもしれません。

aktnk commented 3 years ago

@hrsano645 さん 確認ありがとうございました。

yarnかnpmかは、どちらでもよさそうです。メトロ版を追いかけて同様に依存関係の解消が出来そうなときにはワークフローもメトロ側に合わせるでいいかもしれません。

そうですね。 Webサイトの見た目は気にしてましたが、その裏の環境面は抜けていました。