Skip to content
This repository was archived by the owner on Jun 13, 2023. It is now read-only.

[ruby] Upgrade rubocop to version 1.41.1#321

Closed
depfu[bot] wants to merge 1 commit into
mainfrom
depfu/update/rubocop-1.41.1
Closed

[ruby] Upgrade rubocop to version 1.41.1#321
depfu[bot] wants to merge 1 commit into
mainfrom
depfu/update/rubocop-1.41.1

Conversation

@depfu

@depfu depfu Bot commented Dec 23, 2022

Copy link
Copy Markdown
Contributor

Here is everything you need to know about this upgrade. Please take a good look at what changed and the test results before merging this pull request.

What changed?

✳️ rubocop (~> 1.40.0 → ~> 1.41.1) · Repo · Changelog

Release Notes

1.41.1

Bug fixes

  • #11293: Fix a false negative for Style/Documentation when using macro. (@koic)
  • #11313: Fix a false positive for Naming/BlockForwarding when the block argument is reassigned. (@fatkodima)
  • #11014: Fix a false positive for Style/Aliascop when alias in a method def. (@ydah)
  • #11309: Fix a false positive for Style/RedundantStringEscape when using a redundant escaped string interpolation \#\{foo}. (@koic)
  • #11307: Fix an error for Style/GuardClause when using lvar as an argument of raise in else branch. (@ydah)
  • #11308: Fix disabling departments via comment. (@fatkodima)

Changes

  • #11312: Mark Style/ConcatArrayLiterals as unsafe. (@koic)

Does any of this look wrong? Please let us know.

Commits

See the full diff on Github. The new version differs by 13 commits:


Depfu Status

Depfu will automatically keep this PR conflict-free, as long as you don't add any commits to this branch yourself. You can also trigger a rebase manually by commenting with @depfu rebase.

All Depfu comment commands
@​depfu rebase
Rebases against your default branch and redoes this update
@​depfu recreate
Recreates this PR, overwriting any edits that you've made to it
@​depfu merge
Merges this PR once your tests are passing and conflicts are resolved
@​depfu close
Closes this PR and deletes the branch
@​depfu reopen
Restores the branch and reopens this PR (if it's closed)
@​depfu pause
Ignores all future updates for this dependency and closes this PR
@​depfu pause [minor|major]
Ignores all future minor/major updates for this dependency and closes this PR
@​depfu resume
Future versions of this dependency will create PRs again (leaves this PR as is)

@depfu depfu Bot added the depfu label Dec 23, 2022
@codecov

codecov Bot commented Dec 23, 2022

Copy link
Copy Markdown

Codecov Report

Base: 100.00% // Head: 87.87% // Decreases project coverage by -12.12% ⚠️

Coverage data is based on head (6fa2252) compared to base (26de50f).
Patch has no changes to coverable lines.

Additional details and impacted files
@@             Coverage Diff              @@
##              main     #321       +/-   ##
============================================
- Coverage   100.00%   87.87%   -12.13%     
============================================
  Files            5        5               
  Lines           33       33               
============================================
- Hits            33       29        -4     
- Misses           0        4        +4     
Impacted Files Coverage Δ
app/models/concerns/warclight/solr_document.rb 71.42% <0.00%> (-28.58%) ⬇️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@depfu

depfu Bot commented Jan 2, 2023

Copy link
Copy Markdown
Contributor Author

Closed in favor of #323.

@depfu depfu Bot closed this Jan 2, 2023
@depfu depfu Bot deleted the depfu/update/rubocop-1.41.1 branch January 2, 2023 16:51
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

0 participants