Skip to content

Add Friendly Robber game option#366

Merged
bcollazo merged 4 commits intomasterfrom
friendly-robber
Mar 22, 2026
Merged

Add Friendly Robber game option#366
bcollazo merged 4 commits intomasterfrom
friendly-robber

Conversation

@bcollazo
Copy link
Copy Markdown
Owner

Summary

  • add a Friendly Robber game option in the core library and CLI
  • expose Friendly Robber through the web API and HomePage UI
  • fix robber action pruning when friendly robber removes enemy-owned robber targets

Testing

  • uv run pytest tests/test_game.py tests/models/test_actions.py -q
  • direct CLI smoke test with --config-friendly-robber

@netlify
Copy link
Copy Markdown

netlify bot commented Mar 22, 2026

Deploy Preview for catanatron-staging ready!

Name Link
🔨 Latest commit 1300fc0
🔍 Latest deploy log https://app.netlify.com/projects/catanatron-staging/deploys/69c03732c9ed2500081fafe1
😎 Deploy Preview https://deploy-preview-366--catanatron-staging.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@coveralls
Copy link
Copy Markdown

coveralls commented Mar 22, 2026

Pull Request Test Coverage Report for Build 23409872998

Details

  • 29 of 30 (96.67%) changed or added relevant lines in 6 files are covered.
  • 2 unchanged lines in 2 files lost coverage.
  • Overall coverage increased (+1.4%) to 93.9%

Changes Missing Coverage Covered Lines Changed/Added Lines %
catanatron/catanatron/web/api.py 2 3 66.67%
Files with Coverage Reduction New Missed Lines %
catanatron/catanatron/players/minimax.py 1 96.35%
catanatron/catanatron/players/tree_search_utils.py 1 94.59%
Totals Coverage Status
Change from base Build 23407636985: 1.4%
Covered Lines: 3279
Relevant Lines: 3492

💛 - Coveralls

@bcollazo bcollazo merged commit 82aae93 into master Mar 22, 2026
11 checks passed
@bcollazo bcollazo deleted the friendly-robber branch March 22, 2026 18:54
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