Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@IllianiBird
Copy link
Collaborator

Changed the resources variable back from static to instance-level.

Changed the `resources` variable from static to instance-level to improve encapsulation. This ensures that each dialog can manage its own resources independently, avoiding potential conflicts in shared states.
@IllianiBird IllianiBird requested a review from Scoppio January 9, 2025 23:18
@codecov
Copy link

codecov bot commented Jan 9, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 28.60%. Comparing base (e85cf17) to head (82ffc57).
Report is 4 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff            @@
##             master    #6384   +/-   ##
=========================================
  Coverage     28.59%   28.60%           
  Complexity    14088    14088           
=========================================
  Files          2784     2784           
  Lines        273079   273079           
  Branches      48380    48380           
=========================================
+ Hits          78100    78101    +1     
+ Misses       190999   190997    -2     
- Partials       3980     3981    +1     

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

@IllianiBird IllianiBird merged commit c1c915a into MegaMek:master Jan 9, 2025
6 checks passed
@IllianiBird IllianiBird deleted the restore_nonStatic branch March 23, 2025 22:41
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.

1 participant