Skip to content
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

Unify RuboCop configuration with other Cucumber projects #938

Merged
merged 6 commits into from
Oct 20, 2024
Merged

Conversation

mvz
Copy link
Contributor

@mvz mvz commented Oct 20, 2024

Summary

Update the RuboCop configuration to match other Cucumber projects. This is not an exact science, since not all RuboCop configurations are identical.

Details

  • Switch to default RuboCop StringLiterals style
  • Switch to default RuboCop PercentLiteralDelimiters style
  • Switch to default RuboCop SymbolArray style
  • Switch to default RuboCop LineContinuationLeadingSpace style
  • Re-order RuboCop configuration file
  • Switch to Cucumber standard MessageSpies cop configuration

Motivation and Context

This helps contributors to Cucumber contribute to Aruba without having to worry about having to adhere to different RuboCop rules. See #935 (comment)

How Has This Been Tested?

I ran RuboCop and the changed spec files.

Types of changes

  • Internal change (refactoring, test improvements, developer experience or update of dependencies)

mvz added 6 commits August 31, 2024 21:32
This brings Aruba more in line with other Cucumber Ruby code.
This brings Aruba more in line with other Cucumber Ruby code.
This brings Aruba more in line with other Cucumber Ruby code.
This replace #have_received style mocking with #received style, which is
the style used in other Ruby code in the Cucumber project
@mvz mvz enabled auto-merge October 20, 2024 07:31
@mvz mvz merged commit d98df76 into main Oct 20, 2024
18 checks passed
@mvz mvz deleted the unify-rubocop branch October 20, 2024 08:23
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