Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

[WIP]Update ruby 3.0.3 #3458

Closed
wants to merge 6 commits into from
Closed

[WIP]Update ruby 3.0.3 #3458

wants to merge 6 commits into from

Conversation

ima1zumi
Copy link
Member

No description provided.

@ima1zumi ima1zumi changed the base branch from main to update-stripe October 27, 2021 13:17
Base automatically changed from update-stripe to main November 13, 2021 15:50
@komagata
Copy link
Member

📝 現状rails db:seedが落ちる模様

@ima1zumi
Copy link
Member Author

📝 Ruby 3.0.3 released https://www.ruby-lang.org/ja/news/2021/11/24/ruby-3-0-3-released/

@ima1zumi ima1zumi changed the title [WIP]Update ruby 3.0.2 [WIP]Update ruby 3.0.3 Nov 29, 2021
@ima1zumi
Copy link
Member Author

LintとTestが実行前に以下のエラーでこけてる。
多分bootsnapのバージョンを上げればOK

/home/runner/work/bootcamp/bootcamp/vendor/bundle/ruby/3.0.0/gems/bootsnap-1.7.2/lib/bootsnap/compile_cache/iseq.rb:13:in `to_binary': wrong argument type false (expected Symbol) (TypeError)

Shopify/bootsnap#378

@ima1zumi
Copy link
Member Author

ima1zumi commented Dec 8, 2021

sorcery のバージョンを上げないといけないっぽい

https://github.com/Sorcery/sorcery/issues/270

Run bundle exec rails db:setup
Created database 'ci_test'
rails aborted!
NoMethodError: undefined method `after_save' for {:if=>#<Proc:0x000055a655d84658 /home/runner/work/bootcamp/bootcamp/vendor/bundle/ruby/3.0.0/gems/sorcery-0.15.0/lib/sorcery/model.rb:69>}:Hash

   10  module ActiveSupport
   62    module Callbacks
  142      private
  387        class CallTemplate # :nodoc:
  424          def make_lambda
  425            lambda do |target, value, &block|
  426              target, block, method, *arguments = expand(target, value, block)
❯ 427              target.send(method, *arguments, &block)
  428            end
  429          end
  468        end
  861    end
  862  end

/home/runner/work/bootcamp/bootcamp/vendor/bundle/ruby/3.0.0/gems/activesupport-6.1.4.1/lib/active_support/callbacks.rb:427:in `block in make_lambda'

@ima1zumi
Copy link
Member Author

ima1zumi commented Dec 8, 2021

↑コメントの例外は sorcery 1.6.x に上げれば解決する。
しかし、 sorcery-jwt が sorcery < 1.6 を使うようになっているので上げられない。

https://github.com/hayfever/sorcery-jwt を見ると、 sorcery core に JWT が入るかも?とあるので、 sorcery 単体で jwt を使えるようになっているか調査して、使えるのであれば sorcery-jwt を外して既存コードを書き直す必要がありそう。

@ima1zumi
Copy link
Member Author

ima1zumi commented Jan 3, 2022

sorcery core には JWT は入っていない。検討中で止まっている模様。 https://github.com/Sorcery/sorcery-rework/issues/9
sorcery-jwt を使う限り、 sorcery を 1.6.x にアップデートできず、 Ruby 3.0 にアップデートできない。
この問題は別で issue に立てることにする。

@komagata
Copy link
Member

@ima1zumi こちらテスト通りそうな気がしたので作業してみています。
また、sorcery-jwtにPRを送ってみました。
hayfever/sorcery-jwt#14

@komagata
Copy link
Member

komagata commented Jan 18, 2022

📝 落ちるテスト、mainブランチでも問題なのでmainに向けてPR作って修正していっております〜。
#3994

@komagata
Copy link
Member

📝 3.0.3だとテストが落ちる不具合が 3.1.0だと起きないのでちゃんと調べて直すか悩み中。

@ima1zumi
Copy link
Member Author

ima1zumi commented Jan 26, 2022

Rails 6.1 は Ruby 3.1 に対応してそうなので、3.1 にあげてもいいかもです。(他のGemに不具合が起きなければ)
https://discuss.rubyonrails.org/t/is-rails-6-1-x-compatible-with-ruby-3-1/79713/5

@komagata
Copy link
Member

@ima1zumi

Rails 6.1 は Ruby 3.1 に対応してそうなので、3.1 にあげてもいいかもです。(他のGemに不具合が起きなければ)
https://discuss.rubyonrails.org/t/is-rails-6-1-x-compatible-with-ruby-3-1/79713/5

なるほどですね。

ではruby3.1にする方向で作業進めますね。

@komagata
Copy link
Member

komagata commented Feb 1, 2022

@ima1zumi こちらに3.1.0用のPRを作りました〜。#4108

@komagata
Copy link
Member

komagata commented Feb 4, 2022

ruby 3.1.0のブランチがマージされたのでCloseしちゃいますね。

@komagata komagata closed this Feb 4, 2022
@komagata komagata deleted the update-ruby-3.0.2 branch February 4, 2022 17:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants