-
-
Notifications
You must be signed in to change notification settings - Fork 402
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
Apply new rubocop rules on erb - Layout/ArgumentAlignment #11707
Conversation
8164661
to
07afb55
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍🏽 good work! I knew that there was something wrong somewhere with this indentation rule. Glad to see it applied consistently for once!
@andreslucena I had to merge with the latest develop to fix the conflicts. |
As we've talked privately, lets leave this PR in stand by. If we merge this (and the different new rules) we will have lots of conflicts and currently we have 40 open PRs. We should lower this number until 10 or so, so the number of conlicts is lower too. |
beb3175
to
7f9593d
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I know this is a draft, but I could not help it 😄
LGTM, let me know if you want to introduce something else
Merging as per @andreslucena 's approval. |
* develop: (30 commits) Add `process-content` to erb-lint's deprecated classes (#11762) Add possibility of overriding the tailwind.config.js (#11763) Ask old password when changing email or password (#11737) Add Projects (Budgets) to filtered search (#11740) Fix missing results on Geocoded when search without diacritics (#11761) Add robots.txt instructions (#11693) Add missing activerecord budget locales for search (#11766) Improve design of Admin's Sidebar pages (#11759) Show small static map on admin's meetings index with big screens (#11715) Remove "Manage" button when there's a Sidebar (#11717) Fix admin breadcrumb in Process (#11757) Apply new rubocop rules on erb - Layout/MultilineMethodCallIndentation (#11756) Remove xlarge-* references from admin forms (#11712) Apply new rubocop rules on erb - Argument identation (#11707) Update HERE API autocomplete (#11507) Admin redesign proposal issues (#11668) Redesign: responsive links on cards (#11538) Refactor CI pipelines (#11196) Update postcss and graphql to latest versions (#11733) Fix develop pipeline (#11750) ...
🎩 What? Why?
This PR Attempts to add more rubocop rules to erb linter. This was first highlighted by @andreslucena in #11696 (comment)
Testing