Skip to content

ci: enforce rustdoc warnings and test workspace - #151

Merged
Busiii-adetiba merged 1 commit into
SynapsVault:mainfrom
darcszn:feature/ci-workspace-tests-rustdoc-warnings
Oct 18, 2026
Merged

Busiii-adetiba merged 1 commit into
SynapsVault:mainfrom
darcszn:feature/ci-workspace-tests-rustdoc-warnings

Conversation

@darcszn

@darcszn darcszn commented Oct 18, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Add a workspace-wide cargo test --locked --workspace job alongside the existing per-package contract tests.
  • Require both package-level and workspace-wide test jobs to pass before the WASM build runs.
  • Keep the existing rustdoc warning enforcement: the docs job already sets RUSTDOCFLAGS: -D warnings for cargo doc.

Checklist

  • cargo fmt --all -- --check, cargo clippy --workspace --all-targets -- -D warnings and cargo test --workspace pass (not run as requested)
  • New behaviour is covered by tests (including the unauthorised / error path) (CI-only change; no runtime behaviour changed)
  • docs/CONTRACTS.md and rustdoc updated for any interface, storage, event or error change (not applicable; no interface changes)
  • Error codes were appended, not renumbered (not applicable; no error codes changed)

Validation: Tests and CI checks were not run, and dependencies were not installed, as requested.

closes #118
closes #119

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@drips-wave

drips-wave Bot commented Oct 18, 2026

Copy link
Copy Markdown

@darcszn 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

@Busiii-adetiba
Busiii-adetiba merged commit 24f07d5 into SynapsVault:main Oct 18, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

2 participants