Skip to content

Conversation

@zendesk-edytaroz
Copy link
Contributor

@zendesk-edytaroz zendesk-edytaroz commented Aug 6, 2024

This PR removes the upper boundary on the rails requirement. It is done for the purpose of creating a workflow that tests the library with the current version of rails.
After having done that, it turned out that in the new versions of ActiveRecord some methods had been relocated from ActiveRecord::ConnectionAdapters::AbstractAdapter to ActiveRecord::ConnectionAdapters::ConnectionPool rails/rails@a918394

@zendesk-edytaroz zendesk-edytaroz force-pushed the edytaroz/rails_main_testing branch from 8f99564 to 45c3691 Compare August 6, 2024 12:56
@zendesk-edytaroz zendesk-edytaroz force-pushed the edytaroz/rails_main_testing branch from 45c3691 to 26031b1 Compare August 6, 2024 12:58
@zendesk-edytaroz zendesk-edytaroz marked this pull request as ready for review August 12, 2024 07:26
@zendesk-edytaroz zendesk-edytaroz requested a review from a team August 12, 2024 07:26
@zendesk-edytaroz zendesk-edytaroz force-pushed the edytaroz/rails_main_testing branch 2 times, most recently from ac932e3 to 7aba9e8 Compare August 12, 2024 14:00
@zendesk-edytaroz zendesk-edytaroz force-pushed the edytaroz/rails_main_testing branch from 7aba9e8 to 99cb17e Compare August 13, 2024 09:30
@zendesk-edytaroz zendesk-edytaroz merged commit 8dd4ee1 into master Aug 13, 2024
@zendesk-edytaroz zendesk-edytaroz deleted the edytaroz/rails_main_testing branch August 13, 2024 09:50
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.

3 participants