Skip to content

Commit

Permalink
yjit is automatically activated in rails 7.2
Browse files Browse the repository at this point in the history
  • Loading branch information
manuelmeurer committed Jun 4, 2024
1 parent c9452e5 commit a58c00a
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 12 deletions.
12 changes: 0 additions & 12 deletions app/services/github_notifications/process.rb
Original file line number Diff line number Diff line change
Expand Up @@ -22,18 +22,6 @@ def do_process

return unless @github_notification.completed?

if Date.current > Date.new(2024, 6, 1)
ReportError.call "remove this!"
end
if compat.invalid?
compats = Compat.where(dependencies: compat.dependencies)
if compats.size == RailsRelease.count && !compats.include?(compat)
compat.destroy
@github_notification.destroy
return
end
end

case @github_notification.conclusion
when "success" then status = :compatible
when 'failure' then status = :incompatible
Expand Down
3 changes: 3 additions & 0 deletions config/application.rb
Original file line number Diff line number Diff line change
Expand Up @@ -34,6 +34,9 @@ class Application < Rails::Application
protocol: "https"
}

if Rails.version >= "7.2"
raise "this is not needed anymore, yjit should be enabled by default in rails 7.2."
end
config.after_initialize do
RubyVM::YJIT.enable
end
Expand Down

0 comments on commit a58c00a

Please sign in to comment.