Skip to content

LT-22572: Add IsDisposed check - #1026

Merged
mark-sil merged 1 commit into
mainfrom
LT-22572a
Jul 23, 2026
Merged

LT-22572: Add IsDisposed check#1026
mark-sil merged 1 commit into
mainfrom
LT-22572a

Conversation

@mark-sil

@mark-sil mark-sil commented Jul 23, 2026

Copy link
Copy Markdown
Contributor

This change also fixes these defects:
LT-22573
LT-22574

The regression was introduced with Commit fd245f0.


This change is Reviewable

This change also fixes these defects:
LT-22573
LT-22574

The regression was introduced with Commit fd245f0.
@github-actions

Copy link
Copy Markdown

NUnit Tests

    1 files  ±0      1 suites  ±0   10m 57s ⏱️ -12s
4 313 tests ±0  4 240 ✅ ±0  73 💤 ±0  0 ❌ ±0 
4 322 runs  ±0  4 249 ✅ ±0  73 💤 ±0  0 ❌ ±0 

Results for commit 86e7c41. ± Comparison against base commit 7176e6c.

@codecov-commenter

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 0% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 33.02%. Comparing base (7176e6c) to head (86e7c41).

Files with missing lines Patch % Lines
Src/xWorks/XWorksViewBase.cs 0.00% 0 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1026      +/-   ##
==========================================
- Coverage   33.02%   33.02%   -0.01%     
==========================================
  Files        1202     1202              
  Lines      278232   278232              
  Branches    37169    37169              
==========================================
- Hits        91889    91887       -2     
  Misses     158495   158495              
- Partials    27848    27850       +2     
Files with missing lines Coverage Δ
Src/xWorks/XWorksViewBase.cs 30.40% <0.00%> (-0.24%) ⬇️

... and 1 file with indirect coverage changes

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

@jasonleenaylor jasonleenaylor left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

:lgtm:

@jasonleenaylor reviewed 1 file and all commit messages, and made 1 comment.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on mark-sil).

@mark-sil
mark-sil merged commit 321ba87 into main Jul 23, 2026
7 checks passed
@mark-sil
mark-sil deleted the LT-22572a branch July 23, 2026 17:57
jasonleenaylor pushed a commit that referenced this pull request Jul 23, 2026
This change also fixes these defects:
LT-22573
LT-22574

The regression was introduced with Commit fd245f0.

(cherry picked from commit 321ba87)
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