[ruby] Update bullet 8.1.1 → 8.1.2 (patch)

[depfu]

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

What changed?

✳️ bullet (8.1.1 → 8.1.2) · Repo · Changelog

Release Notes

8.1.2 (from changelog)

• Skip N+1 detection for optional polymorphic belongs_to whose *_type column is nil. ActiveRecord short-circuits the reader to nil without issuing SQL, so the access cannot represent an N+1 query and preloading would be a no-op.
• Fix Set#<< corruption in UnusedEagerLoading#add_eager_loadings split branch

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

Commits

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

• Bumping version to 8.1.2
• Merge pull request #773 from sloane/sloane/bullet-nil-polymorphic-false-positive
• Skip N+1 detection for polymorphic belongs_to with nil _type
• 📝 docs: clarify test wrapping requirements for Bullet
• Merge pull request #771 from martingjaldbaek/fix-add-eager-loadings-set-merge
• Fix Set#<< corruption in UnusedEagerLoading#add_eager_loadings split branch
• Add failing regression test for add_eager_loadings split branch

---

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 cancel merge

Cancels automatic merging of this PR

@​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)

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 87.33%. Comparing base (6bdbfe3) to head (c323ce8).

Additional details and impacted files

@@             Coverage Diff             @@
##           develop    #5830      +/-   ##
===========================================
+ Coverage    87.31%   87.33%   +0.02%     
===========================================
  Files         1480     1480              
  Lines        33637    33637              
  Branches      3569     3569              
===========================================
+ Hits         29369    29378       +9     
+ Misses        4247     4238       -9     
  Partials        21       21              

Flag	Coverage Δ
javascript	76.31% <ø> (ø)
ruby	87.34% <ø> (+0.03%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.