search

codeforjapan / mapprint

80 stars 58 forks source link

Feature/upgrade-version #424

Closed halsk closed 10 months ago

halsk commented 10 months ago

概要 | About

利用しているライブラリのバージョンをアップグレード中。Vue3 への変更には時間がかかり過ぎそうなので、一旦 Vue2 の最新版にアップする

halsk commented 10 months ago

あと、GitHub Actions のチェックも失敗している

halsk commented 10 months ago

以下を除いて、ほとんどのLintエラーは修正完了。

/Users/hal/Documents/workspace/mapprint/components/PrintableMap.vue
  264:13  error  Unexpected mutation of "mapConfig" prop  vue/no-mutating-props

✖ 1 problem (1 error, 0 warnings)

コンポーネントから mapConfig の中身を変更しているのだが、それはバッドプラクティスなんでだめだと。 ちゃんとリファクタリングしなくてはいけない状況・・・

halsk commented 10 months ago

とりあえずメジャーなエラーは潰したのでマージする。ただし、sass のWarningは引き続き出る状況