Skip to content

Deterministic Discards#374

Closed
bcollazo wants to merge 10 commits intomainfrom
bryan/deterministic-discard
Closed

Deterministic Discards#374
bcollazo wants to merge 10 commits intomainfrom
bryan/deterministic-discard

Conversation

@bcollazo
Copy link
Copy Markdown
Owner

This PR builds on #362. It changes the internal representation of DISCARD_RESOURCE actions to just a single resource. This simplifies handling as arrays. It also changes the Frontend to ask for all discards in one-go, and then presents them to the backend one at a time. Among other smaller non-funcional changes.

@bcollazo bcollazo closed this Mar 31, 2026
@netlify
Copy link
Copy Markdown

netlify bot commented Mar 31, 2026

Deploy Preview for catanatron-staging ready!

Name Link
🔨 Latest commit f4c899f
🔍 Latest deploy log https://app.netlify.com/projects/catanatron-staging/deploys/69cb27f0ee373500083ff0ea
😎 Deploy Preview https://deploy-preview-374--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 31, 2026

Pull Request Test Coverage Report for Build 23776569054

Details

  • 42 of 42 (100.0%) changed or added relevant lines in 5 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.05%) to 93.951%

Totals Coverage Status
Change from base Build 23773265000: 0.05%
Covered Lines: 3293
Relevant Lines: 3505

💛 - Coveralls

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.

3 participants