test: add Stellar scalar property coverage#15
Conversation
|
Heads up — we've just set up a Going forward, please branch from and target |
|
All 8 properties from the issue spec are implemented (with the chi-square view-tag test as a nice touch), |
|
Hmm, my earlier merge call failed: conflicts surfaced after #14 landed (likely git fetch origin
git rebase origin/develop
git push --force-with-lease |
|
The 8 properties + CI nightly + WRAITH_FUZZ_RUNS env knob are still the right shape for issue #3. Just two small conflicts now (everything else 3-way merges cleanly):
These came from #14 (JSDoc) and #16 (range scans) extending the same README sections and CI matrix. git fetch origin
git rebase origin/develop
git push --force-with-leaseQuick to resolve. Note: there is a competing PR #46 from @TheDEV111 also targeting issue #3 that I'd close as superseded as soon as yours lands. So no rush from a duplication standpoint, just whoever rebases first. Thanks @kaycke1337. |
|
Closing as superseded. Issue #3 was just resolved by #46 from @TheDEV111, which addressed the same scope and rebased cleanly first. Sorry for the duplicate-effort situation, that's on us for not coordinating contributor assignments tighter. Plenty of open Stellar Wave issues you'd be a great fit for: https://github.com/wraith-protocol/sdk/issues?q=is%3Aopen+label%3A%22Stellar+Wave%22 Thanks for the work @kaycke1337. |
Closes #3
Summary
Validation
Commit used --no-verify because the local hook calls a global pnpm binary that is not installed in this environment; the same commands were run through npx pnpm.