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

Fix RuboCop errors #692

Merged
merged 2 commits into from
May 5, 2024
Merged

Conversation

mmx900
Copy link
Contributor

@mmx900 mmx900 commented May 5, 2024

Context

Fixed RuboCop errors that occurred after #689. These errors seem to be caused by upgrading rubocop-* gems. For example, rubocop-rspec, which was 2.13.2, added RSpec/Rails/InferredSpecType since 2.14.

Summary of Changes

Checklist

  • Tested Mobile Responsiveness
  • Added Unit Tests
  • CI Passes
  • Deploys to Heroku on test Correctly (Maintainers will handle)
  • Added Documentation (Service and Code when required)

Screenshots

Before

After

Copy link
Contributor

@DeeDeeG DeeDeeG left a comment

Choose a reason for hiding this comment

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

Nice, on this branch the Rubocop checks are passing, whereas in #691 the specs themselves were made passing.

The changes here look fairly trivial in terms of not changing the meaning/result of running the code, so, looks good to me!

(In more thoroughness/detail: Same number of "examples" being run in the specs, FWIW, just to confirm this doesn't change the meaning of the specs code. Non-spec code changes are 100% trivial in terms of meaningful changes, just satisfying the lints, so I am not worried about "uncaught" code changes in the app code itself. The lint messages themselves assure that the requested spec code changes are equivalent/were simply redundant or overly verbose before in most cases. So I am confident enough in the (very small) spec refactors as well.)

We may (we should) have green/passing CI again on develop branch with this merged.

Thank you!

@DeeDeeG DeeDeeG merged commit 994fe48 into RefugeRestrooms:develop May 5, 2024
1 check failed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants