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.

Sign up for GitHub

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

Jump to bottom

Removed unused function param #42565

Merged

rafaelfranca merged 1 commit into rails:main from alkesh26:remove-unused-param-from-function

Jun 22, 2021

Contributor

alkesh26 commented Jun 22, 2021

update_sql param got introduced in #41933 (comment). But seems like we are not using / passing it within the function.


          removed unused function param

7463afa

rails-bot bot added the activerecord label

rafaelfranca merged commit e4b1e97 into rails:main

alkesh26 deleted the remove-unused-param-from-function branch

June 23, 2021 06:33

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment