8000 Fixed Minimum Bid Calculation in Mercenary Auction by IllianiBird · Pull Request #6500 · MegaMek/mekhq · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Fixed Minimum Bid Calculation in Mercenary Auction #6500

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

Merged
merged 1 commit into from
Mar 31, 2025

Conversation

IllianiBird
Copy link
Collaborator
  • Corrected the minimum bid logic by removing its dependency on support points, ensuring it now scales accurately with the required combat teams.
  • Adjusted the final bid computation to account for the updated minimum bid calculation.

Dev Notes

Marking this as 'high' as it prevented use of this system.

- Corrected the minimum bid logic by removing its dependency on support points, ensuring it now scales accurately with the required combat teams.
- Adjusted the final bid computation to account for the updated minimum bid calculation.
@IllianiBird IllianiBird added Bug Severity: High Issues described as high severity as per the new issue form labels Mar 30, 2025
@IllianiBird IllianiBird self-assigned this Mar 30, 2025
Copy link
codecov bot commented Mar 30, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 11.29%. Comparing base (cac7cdc) to head (b973989).
Report is 274 commits behind head on master.

Additional details and impacted files
@@             Coverage Diff              @@
##             master    #6500      +/-   ##
============================================
- Coverage     11.29%   11.29%   -0.01%     
+ Complexity     6291     6284       -7     
============================================
  Files          1072     1072              
  Lines        136567   136567              
  Branches      21119    21119              
============================================
- Hits          15430    15419      -11     
- Misses       119592   119604      +12     
+ Partials       1545     1544       -1     

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

@IllianiBird IllianiBird merged commit c31ccbb into MegaMek:master Mar 31, 2025
6 checks passed
@IllianiBird IllianiBird deleted the mercenaryAuctionBids branch March 31, 2025 03:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Severity: High Issues described as high severity as per the new issue form
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant
0