Skip to content

Close #9119: Added Dialog to Allow Mercenary & Pirate Players to Better Control Starting Location#9122

Merged
HammerGS merged 3 commits into
MegaMek:mainfrom
IllianiBird:startingLocationControl
May 30, 2026
Merged

Close #9119: Added Dialog to Allow Mercenary & Pirate Players to Better Control Starting Location#9122
HammerGS merged 3 commits into
MegaMek:mainfrom
IllianiBird:startingLocationControl

Conversation

@IllianiBird
Copy link
Copy Markdown
Collaborator

Close #9119

Another piece of QoL for our player base.

Currently, we always start these campaigns in a semi-random location.

Now mercenary & pirate players can choose whether to start on their faction's capital or a semi-random location.

@IllianiBird IllianiBird self-assigned this May 27, 2026
@IllianiBird IllianiBird requested a review from a team as a code owner May 27, 2026 19:18
@IllianiBird IllianiBird added the UX User experience label May 27, 2026
@codecov
Copy link
Copy Markdown

codecov Bot commented May 27, 2026

Codecov Report

❌ Patch coverage is 0% with 32 lines in your changes missing coverage. Please review.
✅ Project coverage is 15.21%. Comparing base (f2bf383) to head (81d3c31).
⚠️ Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
...q/gui/dialog/StartingSystemConfirmationDialog.java 0.00% 28 Missing ⚠️
MekHQ/src/mekhq/campaign/Campaign.java 0.00% 4 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff            @@
##               main    #9122   +/-   ##
=========================================
  Coverage     15.20%   15.21%           
- Complexity     9335     9339    +4     
=========================================
  Files          1311     1312    +1     
  Lines        173407   173435   +28     
  Branches      26220    26224    +4     
=========================================
+ Hits          26372    26387   +15     
- Misses       143957   143975   +18     
+ Partials       3078     3073    -5     

☔ 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.

Copy link
Copy Markdown
Collaborator

@Sleet01 Sleet01 left a comment

Choose a reason for hiding this comment

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

LGTM

@HammerGS HammerGS merged commit d7585f1 into MegaMek:main May 30, 2026
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

UX User experience

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[RFE] Non Random starting system

3 participants