Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

新しく環境構築すると NoMethodError: tainted? で落ちる #1396

Closed
muryoimpl opened this issue Nov 18, 2023 · 6 comments
Closed

新しく環境構築すると NoMethodError: tainted? で落ちる #1396

muryoimpl opened this issue Nov 18, 2023 · 6 comments

Comments

@muryoimpl
Copy link
Contributor

muryoimpl commented Nov 18, 2023 •
edited
Loading

Ruby 3.2.2 時点ではすでに tainted? がなくなっているので、liquid 5 にしないとNoMethodError で落ちちゃうっぽい。

/Users/muryoimpl/ghq/github.com/kanazawarb/meetup/vendor/bundle/ruby/3.2.0/gems/liquid-4.0.3/lib/liquid/variable.rb:124:in `taint_check': undefined method `tainted?' for "en-US":String (NoMethodError)

https://github.com/Shopify/liquid/blob/main/History.md#500--2021-01-06 でtaint support が削除されているので 5.0.0 以上にしなければならないっぽい。just-the-docs のバージョンあげると解決するのか？要調査。
@muryoimpl
Copy link
Contributor Author

liquid 4.0.4でとりあえず動くみたい。本家の just-the-docs の依存も4.0.4 になっているので、これで一旦進める。

Gemfile.lock がコミットされていないので、環境別で問題の発生有無がでているのでは？というところに戻ってくるね。

これ、もうGemfile.lock 入れていいんじゃないですかね？just-the-docs側もGemfile.lockを入れたことによってdependabot が動作するように環境が整えられているので、意図しないバージョンのおいてけぼりはなくなるんじゃない？というのと、環境が動かなくなることでkzrbメンバーが使いたくない…となるほうが問題ではないか？と思ってます。

@kiyohara
Copy link
Member

@muryoimpl 同意です。 Gemfile.lock いれていきましょう。
以前から議論にあがりつつ、明確な「入れない理由」があったわけではないと認識してます。
どっちでもいいんだけど、ないほうが若干よい面あるよねみたいな。
けど、現状では実害の方が大きく出ているということだと思うので、縛っていきましょう、という気持ちですね。

@PharaohKJ
Copy link
Member

Gemfile.lock のコミット、バージョン管理は must なご時世ではないでしょうか。入れてしまいましょう。

@muryoimpl
Copy link
Contributor Author

muryoimpl commented Nov 29, 2023 •
edited
Loading

ありがとうございます！Slack ↓でも 🆗 いただいているので、Gemfile.lock 含めるようにしますね。
https://kzrb.slack.com/archives/C02A6KL80/p1701144148735009

Gemfile.lock のPLATFORMS はもしかするとお手元の環境で差分でちゃうかもしれないけど、それは都度対応で。
参考: https://bundler.io/v2.4/man/bundle-lock.1.html#SUPPORTING-OTHER-PLATFORMS

muryoimpl added a commit that referenced this issue Dec 6, 2023
@muryoimpl
Gemfile.lock をリポジトリに追加する
57326b2 
新しい環境を構築しようとした際、Gem のバージョンが安定しないことで
`bundle install` 時に失敗することがあるため、Gemfile.lock をリポジトリ
に導入する。

これまでは最新Gemに追随するためにGemfile.lockをあえて入れずにおく意図
であえて導入していなかったが、環境構築に失敗することによるデメリットと、
just-the-docs 本家が dependabot を導入したことによりjust-the-docs を
更新する限りはGemは更新されていくため、導入に踏み切った。
#1396
muryoimpl added a commit that referenced this issue Dec 6, 2023
@muryoimpl
Gemfile.lock をリポジトリに追加する
f1b71e1 
新しい環境を構築しようとした際、Gem のバージョンが安定しないことで
`bundle install` 時に失敗することがあるため、Gemfile.lock をリポジトリ
に導入する。

これまでは最新Gemに追随するためにGemfile.lockをあえて入れずにおく意図
であえて導入していなかったが、環境構築に失敗することによるデメリットと、
just-the-docs 本家が dependabot を導入したことによりjust-the-docs を
更新する限りはGemは更新されていくため、導入に踏み切った。
#1396
@muryoimpl
Copy link
Contributor Author

muryoimpl commented Dec 6, 2023 •
edited
Loading

#1399 の f1b71e1 でGemfile.lockを導入しました。
.gitignore の記述も削除しています。

↑が取り込めたら当issue閉じますね

muryoimpl added a commit that referenced this issue Jan 15, 2024
@muryoimpl
Gemfile.lock をリポジトリに追加する
09653f2 
新しい環境を構築しようとした際、Gem のバージョンが安定しないことで
`bundle install` 時に失敗することがあるため、Gemfile.lock をリポジトリ
に導入する。

これまでは最新Gemに追随するためにGemfile.lockをあえて入れずにおく意図
であえて導入していなかったが、環境構築に失敗することによるデメリットと、
just-the-docs 本家が dependabot を導入したことによりjust-the-docs を
更新する限りはGemは更新されていくため、導入に踏み切った。
#1396
@muryoimpl
Copy link
Contributor Author

#1399 をマージしたので、Gemfile.lock 入りました！

# for free to join this conversation on GitHub. Already have an account? # to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@muryoimpl @kiyohara @PharaohKJ