Skip to content

Enable Rubocop Layout/SpaceAroundOperators#6236

Merged
aduth merged 4 commits intomainfrom
aduth-space-around-operators
Apr 21, 2022
Merged

Enable Rubocop Layout/SpaceAroundOperators#6236
aduth merged 4 commits intomainfrom
aduth-space-around-operators

Conversation

@aduth
Copy link
Contributor

@aduth aduth commented Apr 21, 2022

Previously: #6202 (comment)

Why: So that developers needn't spend their team leaving stylistic review remarks for the conventionally-adopted style.

**Why**: So that developers needn't spend their team leaving stylistic review remarks for the conventionally-adopted style.

changelog: Internal, Linting, Increase strictness of operator styling lints

def self.randomize?(target_percent, discriminator)
max_sha = (16 ** 64) - 1
max_sha = (16**64) - 1
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This was a bit surprising, but appears to be the expected default (source). It can be configured though, if we'd prefer (docs).

Copy link
Contributor Author

@aduth aduth Apr 21, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm gonna call an audible and reinstate the spaces. The inconsistency of the default behavior is pretty jarring to me. Updated in dd9cce3.

Copy link
Contributor

@zachmargolis zachmargolis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@aduth aduth merged commit db0eb46 into main Apr 21, 2022
@aduth aduth deleted the aduth-space-around-operators branch April 21, 2022 21:00
@jmdembe jmdembe mentioned this pull request Apr 26, 2022
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.

3 participants