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

MML Issue 567: Disallow Commercial Armor for BattleMeks #3485

Merged
merged 1 commit into from
Mar 5, 2022

Conversation

SJuliez
Copy link
Member

@SJuliez SJuliez commented Mar 5, 2022

Fixes MML's 567 by removing Commercial Armor from the available armors for BattleMeks (only allowing them for Industrial Meks). The forum thread linked in the issue seems to have a pretty final statement from xotl on that matter. The actual change is lines 101 and 112, the rest is only a bit of renaming for readability.

@codecov
Copy link

codecov bot commented Mar 5, 2022

Codecov Report

Merging #3485 (b38b313) into master (36b96f9) will not change coverage.
The diff coverage is 0.00%.

Impacted file tree graph

@@            Coverage Diff            @@
##             master    #3485   +/-   ##
=========================================
  Coverage     22.67%   22.67%           
  Complexity     4770     4770           
=========================================
  Files          2081     2081           
  Lines        243767   243767           
  Branches      45591    45591           
=========================================
  Hits          55265    55265           
  Misses       187013   187013           
  Partials       1489     1489           
Impacted Files Coverage Δ
megamek/src/megamek/common/verifier/TestMech.java 0.00% <0.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 36b96f9...b38b313. Read the comment docs.

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.

2 participants