Skip to content
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

Update types #902

Merged
merged 6 commits into from
Oct 2, 2024
Merged

Update types #902

merged 6 commits into from
Oct 2, 2024

Conversation

nodech
Copy link
Contributor

@nodech nodech commented Sep 30, 2024

This PR updates the most basic and least modified types, including those in hd, primitives, coins, covenants, protocol, script, net/packets, utils, and ui.

The goal of this PR is to clean up the types and enable simple reference and "go-to-definition" functionality in the TypeScript Language Server.

Code changes have been kept to an absolute minimum to minimize rebases in other PRs.

The TS JSDoc variation of import syntax is not compatible with standard JSDoc, and this has been handled using the rm-imports plugin from bsdoc.
Note: TSC errors are still present in most places where fixes were applied. However, most of these errors cannot be resolved without more extensive changes, or they result from TypeScript's inability to infer types correctly.

@nodech nodech requested a review from Falci September 30, 2024 10:58
@coveralls
Copy link

coveralls commented Sep 30, 2024

Coverage Status

coverage: 70.033%. remained the same
when pulling 0ce3197 on nodech:update-types
into 5294be7 on handshake-org:master.

@nodech
Copy link
Contributor Author

nodech commented Oct 1, 2024

mempool/claimentry - ClaimEntry .name is not consistent within itself. lib/mining/template BlockClaim.name is not consistent with ClaimEntry. Looking into Proof .data and encoding .name should be string. For now leave w/o modifying the code.

@nodech nodech merged commit 45c6ac1 into handshake-org:master Oct 2, 2024
6 checks passed
@nodech nodech added quick review difficulty - easy cleanup improvement classification patch Release version labels Oct 2, 2024
@nodech nodech deleted the update-types branch October 4, 2024 06:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cleanup improvement classification patch Release version quick review difficulty - easy
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants