-
-
Notifications
You must be signed in to change notification settings - Fork 19
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] Initial Autofix Rubocop work #217
Conversation
Fix said newline offense using AF
NB: @chybz - Something was up here with a run that failed on cmake cmate update It might be nothing, but maybe something to check / monitor: https://github.com/cucumber/messages/actions/runs/8817284910/job/24203303631?pr=217 It does look like it was just a flaky connection timeout (But the other 3 then all got auto cancelled too) |
@luke-hill yeee thanks for the report. I fixed the exact same issue in gherkin end just forgot to fix it here also. |
🤔 What's changed?
A bunch of Layout autofix cops. Also a few other small limited Autofix cops.
I have also fixed 2 individual areas where the jsonschema was adding redundant lines/code smells
⚡️ What's your motivation?
Make the codebase rubocop compliant
🏷️ 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.