Skip to content

Avoid multiline endless abstract methods - #1039

Open
KaanOzkan wants to merge 1 commit into
mainfrom
ko-fix-multiline-abstract-super
Open

KaanOzkan wants to merge 1 commit into
mainfrom
ko-fix-multiline-abstract-super

Conversation

@KaanOzkan

Copy link
Copy Markdown
Contributor

Keep translated abstract methods with wrapped parameters lint-clean. Style/EndlessMethod is enabled by rubocop-shopify, and its default allow_single_line style rejects def ... = super when the definition spans lines. Emit super/end for wrapped headers while retaining = super for single-line headers.

Keep translated abstract methods with wrapped parameters lint-clean. [Style/EndlessMethod](https://docs.rubocop.org/rubocop/latest/cops_style.html#styleendlessmethod) is enabled by `rubocop-shopify`, and its default `allow_single_line` style rejects `def ... = super` when the definition spans lines. Emit `super`/`end` for wrapped headers while retaining `= super` for single-line headers.
@KaanOzkan
KaanOzkan marked this pull request as ready for review October 22, 2026 21:02
@KaanOzkan
KaanOzkan requested a review from a team as a code owner October 22, 2026 21:02

This branch has not been deployed

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant