Skip to content

Patch/ci 510 fixups - #536

Open
Cedarich wants to merge 5 commits into
mainfrom
patch/ci-510-fixups
Open

Patch/ci 510 fixups#536
Cedarich wants to merge 5 commits into
mainfrom
patch/ci-510-fixups

Conversation

@Cedarich

Copy link
Copy Markdown
Contributor

Summary

Provide a clear summary of what this PR changes and why.

Related Issue(s)

Link any related issue(s) using #<issue-number>.
If this PR closes an issue, include Closes #<issue-number>.

Type of Change

  • Feature
  • Fix
  • Refactor
  • Documentation
  • Chore

Description

Describe the implementation details and scope of the changes.

Testing Evidence

Describe how this change was tested, including commands run or test cases executed.
Include links to test outputs, screenshots, or any relevant evidence.

Screenshots

Add screenshots, GIFs, or recordings when UI or visual behavior changed.

Checklist

  • Code builds successfully
  • Tests added/updated where applicable
  • Documentation updated if needed
  • Linked issue referenced
  • Ready for review

Ensure soroban/events.json and soroban/events.txt are on their own lines so they are actually ignored.
Ensure Makefile command lines use real tabs and add a fmt-check target that runs `cargo fmt --all -- --check`.
- Add cargo-llvm-cov coverage reporting and upload artifact
- Make cargo-audit actionable (fail on high/critical advisories)
- Make WASM size warn >100KB and fail >150KB
- Keep toolchain pinned to stable with toolchain 1.91.0
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.

1 participant