Skip to content

fix: handle self keyword in warnings checks, and check these warnings are error-free in consistency#2884

Merged
tstirrat15 merged 2 commits intomainfrom
barakmich/warnings
Feb 5, 2026
Merged

fix: handle self keyword in warnings checks, and check these warnings are error-free in consistency#2884
tstirrat15 merged 2 commits intomainfrom
barakmich/warnings

Conversation

@barakmich
Copy link
Contributor

Description

Testing

References

@barakmich barakmich requested a review from a team as a code owner February 5, 2026 20:06
@github-actions github-actions bot added the area/tooling Affects the dev or user toolchain (e.g. tests, ci, build tools) label Feb 5, 2026
@codecov
Copy link

codecov bot commented Feb 5, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 74.88%. Comparing base (03a3a18) to head (5c158ac).
⚠️ Report is 1 commits behind head on main.

❌ Your project status has failed because the head coverage (74.88%) is below the target coverage (75.00%). You can increase the head coverage or adjust the target coverage.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2884      +/-   ##
==========================================
+ Coverage   74.86%   74.88%   +0.02%     
==========================================
  Files         484      484              
  Lines       57994    57996       +2     
==========================================
+ Hits        43413    43426      +13     
+ Misses      11544    11535       -9     
+ Partials     3037     3035       -2     

☔ 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.

tstirrat15
tstirrat15 previously approved these changes Feb 5, 2026
Copy link
Contributor

@tstirrat15 tstirrat15 left a comment

Choose a reason for hiding this comment

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

LGTM

miparnisari
miparnisari previously approved these changes Feb 5, 2026
@barakmich barakmich dismissed stale reviews from miparnisari and tstirrat15 via 5c158ac February 5, 2026 20:30
@github-actions github-actions bot added the area/dependencies Affects dependencies label Feb 5, 2026
@tstirrat15 tstirrat15 merged commit 286d6e8 into main Feb 5, 2026
44 of 45 checks passed
@tstirrat15 tstirrat15 deleted the barakmich/warnings branch February 5, 2026 20:50
@github-actions github-actions bot locked and limited conversation to collaborators Feb 5, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

area/dependencies Affects dependencies area/tooling Affects the dev or user toolchain (e.g. tests, ci, build tools)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants