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

Enabling the RSpec BeforeAfterAll Rubocop rule #10006

Closed
wants to merge 5 commits into from

Conversation

GarryHurleyJr
Copy link
Contributor

@GarryHurleyJr GarryHurleyJr commented Jun 14, 2024

https://www.rubydoc.info/gems/rubocop-rspec/1.15.0/RuboCop/Cop/RSpec/BeforeAfterAll

What are you trying to accomplish?

The BeforeAfterAll RSpec rule was disabled in some parts of the ecosystme. This PR will enable and fix those scenarios it identifies.

Anything you want to highlight for special attention from reviewers?

Please watch for code duplications or conflicts.

How will you know you've accomplished your goal?

The Rubocop rule will work and the tests will not fai afterwards.

Checklist

  • I have run the complete test suite to ensure all tests and linters pass.
  • I have thoroughly tested my code changes to ensure they work as expected, including adding additional tests for new functionality.
  • I have written clear and descriptive commit messages.
  • I have provided a detailed description of the changes in the pull request, including the problem it addresses, how it fixes the problem, and any relevant details about the implementation.
  • I have ensured that the code is well-documented and easy to understand.

@GarryHurleyJr GarryHurleyJr requested review from a team as code owners June 14, 2024 21:25
@github-actions github-actions bot added L: dotnet:nuget NuGet packages via nuget or dotnet L: dart:pub Dart packages via pub labels Jun 14, 2024
@GarryHurleyJr GarryHurleyJr force-pushed the GarryHurleyJr/enableBeforeAfterAll branch from 5590c14 to fd33f8b Compare June 14, 2024 21:31
.rubocop_todo.yml Outdated Show resolved Hide resolved
@GarryHurleyJr
Copy link
Contributor Author

Some of the rubocops were mixed up in main, so with a merge from main, things got out of sync. This branch should reflect the rubocop status in main.

@jurre
Copy link
Member

jurre commented Jun 18, 2024

@GarryHurleyJr still looks like there are unrelated changes in this PR. Maybe easiest to just do a hard reset to main and re-apply the changes.

@GarryHurleyJr GarryHurleyJr force-pushed the GarryHurleyJr/enableBeforeAfterAll branch from b4d9711 to 6f19702 Compare June 18, 2024 20:49
@GarryHurleyJr GarryHurleyJr reopened this Jun 18, 2024
@GarryHurleyJr
Copy link
Contributor Author

GarryHurleyJr commented Jun 18, 2024

@GarryHurleyJr still looks like there are unrelated changes in this PR. Maybe easiest to just do a hard reset to main and re-apply the changes.

The problem is that PR #9820 re-added some other rubocop rules that had been removed. I have updated that here so that we are at least showing the proper status of rubocop_todo.yml. Alternatively, I coud peel out those changes into a separate PR to merge into main and keep this and other PR's clean.

GarryHurleyJr and others added 2 commits June 18, 2024 18:16
* Strict type Dependabot::Bundler::ChildGemFileFinder.

* Strict type Dependabot::Bundler::ChildGemfileFinder.
@GarryHurleyJr
Copy link
Contributor Author

#10041 contains the code that was muddying this PR.

@GarryHurleyJr
Copy link
Contributor Author

creating a new PR for this issue

@GarryHurleyJr GarryHurleyJr deleted the GarryHurleyJr/enableBeforeAfterAll branch June 19, 2024 16:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
L: dart:pub Dart packages via pub L: dotnet:nuget NuGet packages via nuget or dotnet L: javascript
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants