-
Notifications
You must be signed in to change notification settings - Fork 0
Conversation
感謝ですー |
なんか私に権限がなくてマージできないらしい… |
@h-takeyeah 誰にメンションしたらいいですか?w |
誰か、権限ある方、マージをお願いします(メンション間違ってたらごめんなさい) |
There was a problem hiding this 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", |
There was a problem hiding this comment.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@h-takeyeah bulmaはそれにしましたがeslintが今度はエラーを出しましたので、
ほかもバージョン上げないほうがいいかもしれないかもです。
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🥺
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@h-takeyeah
解決しました。
eslintに破壊的変更があったので、biomeに移行しました。
当初のPRの範囲を超えているのでフォーマッタの変更はするべきでないと考えます。 (私信をここに書いてしまって申し訳ありませんが他に連絡手段がないので→) @negitoro001 別件ですがgmailの方にメールしたんですが読んでいただけましたか? |
@h-takeyeah メールが迷惑メールに入ってたので気づきませんでしたw 返信しましたので確認をお願いします。 確かに、当初のPRの範囲を超えてしまったのですが、 ついでに修正をしてしまいました。
ただ、本来のPRが単なるパッケージのアプデだけなので、それならいっそPRを閉じても良いかもしれません。 長文失礼しました。 |
📣 諸々のやり取りの結果、このリポジトリはアーカイブされることになりました。 アーカイブ前にこのPRを閉じてもらうようお願いしてあります。 |
@h-takeyeah
定期的にアップデートしないとと思ってやってます。動作することはdockerで確認しました
close #66
close #65
close #64
close #67