-
-
Notifications
You must be signed in to change notification settings - Fork 48
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
Ruby: Layout rubocop fixes #235
Conversation
@mpkorstanje whilst doing some very basic refactoring / linting here. I noticed we have a situation in which we want to define a method called I imagine in other languages it's a similar issue. But it would be good if we could maybe change this to something like Given you've done most of the work here recently, other than Aslak I figured I'd ask you for your thoughts. I also noticed that we seem to refer to the iVar |
I've raised an issue here: #236 to discuss this in more detail |
🤔 What's changed?
Bunch of simple layout fixes
⚡️ What's your motivation?
All autofixes, clear a lot of the low-hanging fruit
🏷️ What kind of change is this?
♻️ Anything particular you want feedback on?
📋 Checklist:
This text was originally generated from a template, then edited by hand. You can modify the template here.