Skip to content

Conversation

@harshithad0703
Copy link
Contributor

No description provided.

@harshithad0703 harshithad0703 requested a review from a team as a code owner January 12, 2026 13:23
@github-actions
Copy link

github-actions bot commented Jan 12, 2026

Coverage report

St.
Category Percentage Covered / Total
🟢 Statements 100% 223/223
🟢 Branches 98.29% 115/117
🟢 Functions 100% 34/34
🟢 Lines 100% 207/207

Test suite run success

128 tests passing in 7 suites.

Report generated by 🧪jest coverage report action from 95f925b

@github-actions
Copy link

github-actions bot commented Jan 12, 2026

Coverage report

St.
Category Percentage Covered / Total
🟢 Statements 100% 223/223
🟢 Branches 98.29% 115/117
🟢 Functions 100% 34/34
🟢 Lines 100% 207/207

Test suite run success

128 tests passing in 7 suites.

Report generated by 🧪jest coverage report action from 95f925b

Copy link

@timbenniks timbenniks left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If this works, let's merge. However. I feel like the postbuild.js file is kind of messy. Perhaps we should investigate the voidzero toolchain for this SDK.

aman19K
aman19K previously approved these changes Jan 12, 2026
@aman19K
Copy link

aman19K commented Jan 12, 2026

@nadeem-cs @harshithad0703 , before merging the PR pls resolve this #176 (review)

@nadeem-cs
Copy link
Contributor

@timbenniks Its working for now. We'll go ahead for release with this.

Copy link

@timbenniks timbenniks left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good

@harshithad0703 harshithad0703 merged commit ca18aa6 into main Jan 12, 2026
14 checks passed
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.

5 participants