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

Release 5.0.0 #393

Merged
merged 3 commits into from
Jun 25, 2024
Merged

Release 5.0.0 #393

merged 3 commits into from
Jun 25, 2024

Conversation

AgaDufrat
Copy link
Contributor

@AgaDufrat AgaDufrat commented Jun 18, 2024

We may want to do a pre release to update config for the new cops.

Tested against:

*Failing with previously pending cops are now enabled by default: RSpec/BeEmpty, RSpec/BeEq, RSpec/BeNil, RSpec/ChangeByZero, RSpec/ClassCheck, RSpec/ContainExactly, RSpec/DuplicatedMetadata, RSpec/EmptyMetadata, RSpec/EmptyOutput, RSpec/Eq, RSpec/ExcessiveDocstringSpacing, RSpec/ExpectInLet, RSpec/IdenticalEqualityAssertion, RSpec/IndexedLet, RSpec/IsExpectedSpecify, RSpec/MatchArray, RSpec/MetadataStyle, RSpec/NoExpectationExample, RSpec/PendingWithoutReason, RSpec/ReceiveMessages, RSpec/RedundantAround, RSpec/RedundantPredicateMatcher, RSpec/RemoveConst, RSpec/RepeatedSubjectCall, RSpec/SkipBlockInsideExample, RSpec/SortMetadata, RSpec/SpecFilePathFormat, RSpec/SpecFilePathSuffix, RSpec/SubjectDeclaration, RSpec/UndescriptiveLiteralsDescription, and RSpec/VerifiedDoubleReference

This is a more appropriate place for it after `RSpec/Capybara/FeatureMethods`
cop was migrated to RSpec/Dialect.
so that all Capybara-specific methods that have the same native RSpec method
(e.g. are just aliases) following the upgrade to rubocop-rspec 3.0.

See:
- https://github.com/rubocop/rubocop-rspec/pull/1848/files
- rubocop/rubocop-rspec@1fa9ce3

> If you were previously using the ‘RSpec/Capybara/FeatureMethods` cop and want to keep disabling all Capybara-specific methods that have the same native RSpec method (e.g. are just aliases), use the following config:
>
> ```
> RSpec/Dialect:
>    PreferredMethods:
>      background: :before
>      scenario:   :it
>      xscenario:  :xit
>      given:      :let
>      given!:     :let!
>      feature:    :describe
> ```
@AgaDufrat
Copy link
Contributor Author

@baisa, @ryanb-gds this release enables a lot of new cop, which will be flag up in your apps. Would you like to test the branch first and consider any additional config changes we may want to add as part of this release?

@ryanb-gds
Copy link

ryanb-gds commented Jun 21, 2024

@baisa, @ryanb-gds this release enables a lot of new cop, which will be flag up in your apps. Would you like to test the branch first and consider any additional config changes we may want to add as part of this release?

It's probably not something we're going to have time to do in the next couple of weeks, and I'm not generally a fan of overriding linter configs anyway, so I'm inclined to say "no" and we'll come back to you if there's anything that's going to cause us serious trouble?

[EDIT]: Given Whitehall appears to have not had any major issues I'd imagine it won't be too bad.

@AgaDufrat
Copy link
Contributor Author

Thanks. In such case I'm merging it.

@AgaDufrat AgaDufrat marked this pull request as ready for review June 25, 2024 13:02
@AgaDufrat AgaDufrat merged commit 6f199ed into main Jun 25, 2024
11 checks passed
@AgaDufrat AgaDufrat deleted the new-release branch June 25, 2024 13:02
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