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

Replaces the hash rocket operator in favor of the newer Ruby syntax on render #43368

Merged
merged 1 commit into from
Dec 8, 2021

Conversation

liuriloami
Copy link
Contributor

Summary

Updates the calls to render replacing the hash rocket operator in favor of the new syntax introduced in Ruby 1.9.1 and Ruby 2.2 for code and style consistency.

Other Information

This is a follow up of #43367, trying to update some code that was still using the hash rocket syntax while the majority of the code wasn't.

What I did here was replacing render :([a-z0-9_]+) => by render $1:, while double checking for calls that had more than one param which had to be replaced by hand.

@nvasilevski
Copy link
Contributor

Seems like a reasonable thing to have 👍
Perhaps needs to be rebased

@rafaelfranca
Copy link
Member

We don't accept cosmetic changes but I'll open an exception because it is a first time contributor

@rafaelfranca rafaelfranca merged commit 7160f46 into rails:main Dec 8, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants