-
Notifications
You must be signed in to change notification settings - Fork 32
Static class refactor #140
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
Changes from all commits
Commits
Show all changes
22 commits
Select commit
Hold shift + click to select a range
e2233d2
refactor: replace dynamic xdr construction with static classes
Ryang-21 7f15314
refactor: add tests for new xdr classes / replace mocha-karma-sinon t…
Ryang-21 520e229
feat: modernize build chain and package configs
Ryang-21 d188fc3
refactor: optimize Writer class to use static buffer and ensure capac…
Ryang-21 7525f23
add missing exports
Ryang-21 3ddf1bf
add validateXdr method to XdrType interface and BaseType class
Ryang-21 7b0c982
add detailed JSDoc comments for XDR types and core classes
Ryang-21 fa86cfd
update examples to the new xdr schema
Ryang-21 acd054e
add migration guide and update changelog / readme
Ryang-21 4c20625
remove xdrgen codegen files
Ryang-21 6166c52
remove legacy configs that are no longer used
Ryang-21 309c5ac
update husky to v9
Ryang-21 070a969
expose nameByValue on EnumSchema
Ryang-21 fa8e38d
convert examples to typescript
Ryang-21 138961d
bump node version to 22
Ryang-21 cf85aff
add tests to validate new wire encode/decodes the same
Ryang-21 af7335f
address copilot feedback
Ryang-21 9b7352c
prep for rc.1
Ryang-21 e1db4db
add xdr test object with multi nested structure for compat testing
Ryang-21 e287a55
bump vitest to v4.1.0 and add vite as its no longer bundled with vitest
Ryang-21 15ef1f0
add pnpm workspace and define min release age for dependencies
Ryang-21 3093db2
fix naming of node 20 -> 22 and bump required pnpm version to 10
Ryang-21 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file was deleted.
Oops, something went wrong.
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
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
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,4 +1 @@ | ||
| #!/usr/bin/env sh | ||
| . "$(dirname -- "$0")/_/husky.sh" | ||
|
|
||
| pnpm lint-staged | ||
|
Ryang-21 marked this conversation as resolved.
|
||
This file was deleted.
Oops, something went wrong.
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
This file was deleted.
Oops, something went wrong.
This file was deleted.
Oops, something went wrong.
Oops, something went wrong.
Oops, something went wrong.
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.
Uh oh!
There was an error while loading. Please reload this page.