Skip to content
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

Issue 5373: Fixed fix references not adding gunners #5374

Conversation

psikomonkie
Copy link
Collaborator

@psikomonkie psikomonkie commented Dec 14, 2024

Fixes #5373
Fixes #5372
Fixes #5375

Improperly added & removed gunners originally. Verified after this fix I can load the original save from the issue correctly. I also verified I can make a new save and that it loads properly.

@codecov-commenter
Copy link

codecov-commenter commented Dec 14, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 10.46%. Comparing base (a098581) to head (b78f41e).
Report is 37 commits behind head on master.

Additional details and impacted files
@@             Coverage Diff              @@
##             master    #5374      +/-   ##
============================================
- Coverage     10.47%   10.46%   -0.01%     
+ Complexity     6066     6061       -5     
============================================
  Files           959      959              
  Lines        135598   135600       +2     
  Branches      19754    19756       +2     
============================================
- Hits          14199    14196       -3     
- Misses       120046   120051       +5     
  Partials       1353     1353              

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

@IllianiCBT IllianiCBT merged commit 0f7b985 into MegaMek:master Dec 15, 2024
4 checks passed
IllianiCBT added a commit to IllianiCBT/mekhq that referenced this pull request Dec 15, 2024
…and-vessals-ignoring-personnel-assignments

Issue 5373: Fixed fix references not adding gunners
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants