-
Notifications
You must be signed in to change notification settings - Fork 290
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Autoresolve support #6208
base: master
Are you sure you want to change the base?
Autoresolve support #6208
Conversation
Is this still in Development, also could we get a description so we know what it does. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
See note regarding Description and status
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #6208 +/- ##
=========================================
Coverage 29.01% 29.01%
- Complexity 13982 13984 +2
=========================================
Files 2649 2649
Lines 268128 268128
Branches 47723 47723
=========================================
+ Hits 77798 77801 +3
+ Misses 186445 186440 -5
- Partials 3885 3887 +2 ☔ View full report in Codecov by Sentry. 🚨 Try these New Features:
|
I talked with Illiani, we will move forward with this one, as the other type of autoresolve I am planning is in the back burner and may take longer than expected. |
Extracts interface for some "end of game" events, this small necessary change is necessary for the implementation of an autoresolve feature that integrates with the Resolve Scenario wizard.
MegaMek/mekhq#5155