Skip to content

fix(workspace): correct Cargo workspace members list - #315

Merged
greatest0fallt1me merged 1 commit into
CalloraOrg:mainfrom
MerlinTheWhiz:fix/workspace-cargo-members
Apr 26, 2026
Merged

fix(workspace): correct Cargo workspace members list#315
greatest0fallt1me merged 1 commit into
CalloraOrg:mainfrom
MerlinTheWhiz:fix/workspace-cargo-members

Conversation

@MerlinTheWhiz

Copy link
Copy Markdown
Contributor

Closes #238

Changes made:

  • Cargo.toml: normalized workspace members and default-members so vault, revenue_pool, and settlement appear exactly once.
  • contracts/vault/src/lib.rs: fixed broken vault logic around storage keys, authorized caller handling, settlement checks, allowlist compatibility methods, event payloads, and deduct/batch-deduct behavior.
  • contracts/vault/src/test.rs: updated vault tests/fuzz cases to match the repaired contract behavior and restore passing coverage.
  • coverage/cobertura.xml, coverage/tarpaulin-report.html: regenerated coverage reports after validation.

@MerlinTheWhiz
MerlinTheWhiz force-pushed the fix/workspace-cargo-members branch from ee781d2 to 819a169 Compare April 25, 2026 09:32
@drips-wave

drips-wave Bot commented Apr 25, 2026

Copy link
Copy Markdown

@MerlinTheWhiz Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

@greatest0fallt1me
greatest0fallt1me merged commit 273714a into CalloraOrg:main Apr 26, 2026
0 of 3 checks passed
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.

Workspace: fix duplicate [workspace].members in root Cargo.toml

2 participants