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

Skip to content

Conversation

@IllianiBird
Copy link
Collaborator

  • Switched the 'Land Air' part of LAM to be 'Land Air' and not 'Landair'

The rest is automated formatting changes to bring the class into compliance.

- Added `team` property to `BotForceStub` to support differentiated teams for scenarios.
- Refactored blind drop handling in `AtBScenarioViewPanel` to improve unit representation under specific conditions.
@IllianiBird IllianiBird added Bug Any kind of issue that cannot be fixed without code changes GUI User Interface Severity: Low Issues described as low severity as per the new issue form labels Mar 27, 2025
@IllianiBird IllianiBird self-assigned this Mar 27, 2025
public static String getEntityTypeName(long typeId) {
if ((typeId & ETYPE_LAND_AIR_MEK) == ETYPE_LAND_AIR_MEK) {
return "Landair Mek";
return "Land Air Mek";
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Behold: the sole non-formatting change.

@codecov
Copy link

codecov bot commented Mar 27, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 29.67%. Comparing base (d932184) to head (50392fa).
Report is 12 commits behind head on master.

Additional details and impacted files
@@             Coverage Diff              @@
##             master    #6776      +/-   ##
============================================
- Coverage     29.68%   29.67%   -0.01%     
+ Complexity    15605    15588      -17     
============================================
  Files          2864     2864              
  Lines        280887   280889       +2     
  Branches      49295    49296       +1     
============================================
- Hits          83388    83365      -23     
- Misses       191950   191975      +25     
  Partials       5549     5549              

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

public PilotingRollData checkSkid(EntityMovementType moveType, Hex prevHex, EntityMovementType overallMoveType, MoveStep prevStep, MoveStep currStep, int prevFacing, int curFacing, Coords lastPos, Coords curPos, boolean isInfantry, int distance) {
public PilotingRollData checkSkid(EntityMovementType moveType, Hex prevHex, EntityMovementType overallMoveType,
MoveStep prevStep, MoveStep currStep, int prevFacing, int curFacing,
Coords lastPos, Coords curPos, boolean isInfantry, int distance) {

Check notice

Code scanning / CodeQL

Useless parameter Note

The parameter 'isInfantry' is never used.
@rjhancock rjhancock merged commit 11e0dbf into MegaMek:master Mar 28, 2025
6 checks passed
@IllianiBird IllianiBird deleted the blindDropForRealThisTime branch March 31, 2025 03:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Bug Any kind of issue that cannot be fixed without code changes GUI User Interface Severity: Low Issues described as low severity as per the new issue form

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants