search

hirosystems / stacks.js

JavaScript libraries for identity, auth, storage and transactions on the Stacks blockchain.
https://stacks.js.org
MIT License
944 stars 307 forks source link

refactor!: update clarity representation #1646

Closed janniks closed 4 months ago

janniks commented 4 months ago

This PR was published to npm with the version 6.11.4-pr.e279ac3.0 e.g. npm install @stacks/common@6.11.4-pr.e279ac3.0 --save-exact

build Update commitlint config to allow longer lines (e.g. when adding BREAKING CHANGE body to commit message.

refactor! Introduce a new ClarityWireType to replace the existing ClarityType, which now is human-readable.

This is done in anticipation of https://github.com/stacksgov/sips/pull/166

Next steps

In a future PR, I may update the .buffer, .value etc. to all follow a similar pattern (e.g. to all be .value)

vercel[bot] commented 4 months ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
stacksjs-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Mar 7, 2024 4:01pm
codecov[bot] commented 4 months ago

Codecov Report

Attention: Patch coverage is 76.80412% with 135 lines in your changes are missing coverage. Please review.

Project coverage is 66.28%. Comparing base (922e037) to head (e279ac3). Report is 12 commits behind head on next.

:exclamation: Current head e279ac3 differs from pull request most recent head 6ddc393. Consider uploading reports for the commit 6ddc393 to get more accurate results

Files Patch % Lines
packages/api/src/api.ts 19.40% 54 Missing :warning:
packages/cli/src/cli.ts 40.00% 24 Missing :warning:
packages/transactions/src/fetch.ts 89.74% 12 Missing :warning:
packages/network/src/network.ts 62.06% 8 Missing and 3 partials :warning:
packages/transactions/src/builders.ts 84.78% 7 Missing :warning:
packages/profile/src/profile.ts 25.00% 6 Missing :warning:
packages/stacking/tests/apiMockingHelpers.ts 40.00% 6 Missing :warning:
packages/auth/src/profile.ts 83.33% 4 Missing :warning:
packages/common/src/utils.ts 69.23% 2 Missing and 2 partials :warning:
packages/common/src/fetch.ts 40.00% 3 Missing :warning:
... and 3 more
Additional details and impacted files ```diff @@ Coverage Diff @@ ## next #1646 +/- ## ========================================== - Coverage 66.37% 66.28% -0.09% ========================================== Files 119 124 +5 Lines 8717 8676 -41 Branches 1920 1894 -26 ========================================== - Hits 5786 5751 -35 + Misses 2806 2803 -3 + Partials 125 122 -3 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.