Skip to content
This repository has been archived by the owner on Aug 29, 2024. It is now read-only.

パッケージのアップデート&フォーマッタをESlintからbiomeに移行 #68

Closed
wants to merge 7 commits into from

Conversation

negitoro001
Copy link
Contributor

@negitoro001 negitoro001 commented Aug 5, 2024

@h-takeyeah

定期的にアップデートしないとと思ってやってます。動作することはdockerで確認しました

close #66
close #65
close #64
close #67

@h-takeyeah
Copy link
Contributor

感謝ですー

@h-takeyeah
Copy link
Contributor

h-takeyeah commented Aug 5, 2024

なんか私に権限がなくてマージできないらしい…

@negitoro001
Copy link
Contributor Author

@h-takeyeah 誰にメンションしたらいいですか?w

@negitoro001
Copy link
Contributor Author

negitoro001 commented Aug 20, 2024

誰か、権限ある方、マージをお願いします(メンション間違ってたらごめんなさい)

Copy link
Contributor

@h-takeyeah h-takeyeah left a comment

Choose a reason for hiding this comment

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

マージできないついでに改めて見たら気になるポイントがあったのでコメント 🙇


あと余談ですが

動作することはdockerで確認しました

docker-compose.ymlについて。working_dirを指定してないとcommandのところに書いてあるnpm installをしようとして(/package.jsonを探しに行くので)必ずコケるんじゃない? 🥺

package.json Outdated
"@fortawesome/vue-fontawesome": "^3.0.8",
"@types/node": "^22",
"@vitejs/plugin-vue": "^5.1.2",
"bulma": "^1.0.2",
Copy link
Contributor

Choose a reason for hiding this comment

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

bulmaを1.x.xにアップデートするとbulma-calendarのpeer dependencyが壊れるのでbulmaのバージョンは0.9.4のままの方がいいと思います。

npm error code ERESOLVE
npm error ERESOLVE could not resolve
npm error
npm error While resolving: bulma-calendar@6.1.19
npm error Found: bulma@1.0.2
npm error node_modules/bulma
npm error   dev bulma@"^1.0.2" from the root project
npm error
npm error Could not resolve dependency:
npm error peer bulma@"^0.9.4" from bulma-calendar@6.1.19
npm error node_modules/bulma-calendar
npm error   bulma-calendar@"^6.1.19" from the root project
npm error
npm error Conflicting peer dependency: bulma@0.9.4
npm error node_modules/bulma
npm error   peer bulma@"^0.9.4" from bulma-calendar@6.1.19
npm error   node_modules/bulma-calendar
npm error     bulma-calendar@"^6.1.19" from the root project
npm error
npm error Fix the upstream dependency conflict, or retry
npm error this command with --force or --legacy-peer-deps
npm error to accept an incorrect (and potentially broken) dependency resolution.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@h-takeyeah bulmaはそれにしましたがeslintが今度はエラーを出しましたので、

ほかもバージョン上げないほうがいいかもしれないかもです。

Copy link
Contributor

Choose a reason for hiding this comment

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

🥺

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@h-takeyeah
解決しました。

eslintに破壊的変更があったので、biomeに移行しました。

@negitoro001 negitoro001 changed the title パッケージのアップデート パッケージのアップデート&フォーマッタをESlintからbiomeに移行 Aug 27, 2024
@h-takeyeah
Copy link
Contributor

当初のPRの範囲を超えているのでフォーマッタの変更はするべきでないと考えます。

(私信をここに書いてしまって申し訳ありませんが他に連絡手段がないので→) @negitoro001 別件ですがgmailの方にメールしたんですが読んでいただけましたか?

@negitoro001
Copy link
Contributor Author

@h-takeyeah メールが迷惑メールに入ってたので気づきませんでしたw

返信しましたので確認をお願いします。

確かに、当初のPRの範囲を超えてしまったのですが、 npm install でこけるものを用意するのもよくないと思ったので、

ついでに修正をしてしまいました。

eslint は破壊的変更が加えられていて、残念ながら現状pluginが全滅している状態なので、暫定的にbiomeに変更しました。

ただ、本来のPRが単なるパッケージのアプデだけなので、それならいっそPRを閉じても良いかもしれません。

長文失礼しました。

@h-takeyeah
Copy link
Contributor

📣 諸々のやり取りの結果、このリポジトリはアーカイブされることになりました。

アーカイブ前にこのPRを閉じてもらうようお願いしてあります。

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants