Skip to content

Conversation

@gavvvr
Copy link
Contributor

@gavvvr gavvvr commented Sep 14, 2020

Fix NoMethodError: undefined method ArgumentError' for #ActiveRecord::ConnectionAdapters::OracleEnhancedAdapter:`

Fix `NoMethodError: undefined method `ArgumentError' for #<ActiveRecord::ConnectionAdapters::OracleEnhancedAdapter:>`
Copy link
Collaborator

@koic koic left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice! This looks good to me.

@yahonda This is a regression by #1669. I think it's worth backporting to release60 branch. What do you think about that?

@yahonda
Copy link
Collaborator

yahonda commented Sep 14, 2020

Thanks for the review. I'd need some specs to avoid future regressions.

@yahonda
Copy link
Collaborator

yahonda commented Sep 14, 2020

I mean I'm happy to backport this change to release60 branch with specs.

@yahonda
Copy link
Collaborator

yahonda commented Sep 15, 2020

Nevermind. Merging this pull request.

@yahonda yahonda merged commit dcf841d into rsim:master Sep 15, 2020
yahonda added a commit to yahonda/oracle-enhanced that referenced this pull request Sep 15, 2020
yahonda added a commit that referenced this pull request Sep 16, 2020
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