Fix/workflow issues - #376
Closed
Kureszn wants to merge 6 commits into
Closed
Conversation
…ring migration notes for schema changes
…exChainx#227 - ApexChainx#90: Add release-tests CI job running cargo test --release --lib - ApexChainx#221: Add deterministic concurrency policy tests for calculate_sla (idempotent replay, contradictory input rejection, config-change reset, anti-spam recalc limit enforcement) - ApexChainx#225: Add api_stability module with maintainer-facing stability scoring (canonical field counts, error code enumeration, event symbol stability, storage key stability, and CI-runnable assessment) - ApexChainx#227: Add retryable vs terminal error classification harness (is_terminal classification table, coverage assertions for all 19 SLAError variants, count consistency verification) Closes ApexChainx#90, closes ApexChainx#221, closes ApexChainx#225, closes ApexChainx#227
# Conflicts: # apexchainx_calculator/src/tests.rs
…yml workflow bugs - Add docs/STORAGE_KEY_MIGRATION_CHECKLIST.md with 6-item checklist and decision table for new/removed/renamed storage keys (SC-509) - Add SC-509 section to docs/CONTRACT_MAINTENANCE_POLICY.md with ToC entry and issue cross-reference row - Add SC-509 checkbox to the Smart Contract PR checklist in CONTRIBUTING.md and a brief SC-509 paragraph in the Contract Maintenance Policies section - Add pointer comment to the Storage Keys block in lib.rs directing contributors to the checklist before touching that block CI fixes found during workflow audit: - Remove duplicate dtolnay/rust-toolchain@stable step in client-checks job; the pinned @1.94.1 step that follows it is the one that actually matters - Fix broken WASM path in provenance-hashes Generate hash step: was target/wasm32-unknown-unknown/... (relative to apexchainx_calculator/) but the step runs from repo root, so prefix with apexchainx_calculator/
usmanimamu17-create
requested changes
Jul 30, 2026
Kureszn
force-pushed
the
fix/workflow-issues
branch
from
July 30, 2026 14:09
3e122f6 to
0557060
Compare
Contributor
Author
|
Very sorry about that. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
All issues are fixed and should pass
Closes #271