Skip to content

Code Smells for evm-extension - #984

Merged
gummy789j merged 11 commits into
tronprotocol:feat/architecture-evm-extensionfrom
grayfoxd:feat/architecture-evm-extension
Aug 26, 2026
Merged

Code Smells for evm-extension#984
gummy789j merged 11 commits into
tronprotocol:feat/architecture-evm-extensionfrom
grayfoxd:feat/architecture-evm-extension

Conversation

@grayfoxd

Copy link
Copy Markdown
Contributor

No description provided.

@grayfoxd grayfoxd closed this Aug 25, 2026
@grayfoxd
grayfoxd force-pushed the feat/architecture-evm-extension branch from b4f8463 to f5ee5a8 Compare August 25, 2026 07:05
@grayfoxd grayfoxd reopened this Aug 25, 2026
@grayfoxd grayfoxd closed this Aug 25, 2026
@grayfoxd
grayfoxd force-pushed the feat/architecture-evm-extension branch from 54f4e94 to 44a40ee Compare August 25, 2026 07:59
@grayfoxd grayfoxd reopened this Aug 25, 2026

@gummy789j gummy789j left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

The extractions (tx-build.ts), the safe-integer helpers, the malformed-JSON guard, the required-dependency cleanup and the family-scoped --json-schema fix are all good and I'd like to take them.

@gummy789j
gummy789j merged commit 701c58c into tronprotocol:feat/architecture-evm-extension Aug 26, 2026
2 of 4 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.

2 participants