feat!: delete ApproveToken helper#70
Conversation
BREAKING CHANGE: Deletes the ApproveToken helper and related tests. This was a helper method that was used by legacy changesets which were no longer used.
|
👋 jkongie, thanks for creating this pull request! To help reviewers, please consider creating future PRs as drafts first. This allows you to self-review and make any final changes before notifying the team. Once you're ready, you can mark it as "Ready for review" to request feedback. Thanks! |
There was a problem hiding this comment.
Pull request overview
Removes the legacy ApproveToken helper from pkg/cldfutil/changeset and deletes its associated tests, as it is no longer used by any changesets in this repository (breaking change).
Changes:
- Deleted
ApproveTokenhelper (ERC20 approval helper for EVM chains). - Removed the simulated-EVM unit tests that exercised
ApproveToken.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| pkg/cldfutil/changeset/token_approve.go | Removed the unused ApproveToken helper implementation. |
| pkg/cldfutil/changeset/token_approve_test.go | Removed tests that only validated the deleted helper. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
BREAKING CHANGE: Deletes the ApproveToken helper and related tests.
This was a helper method that was used by legacy changesets which were no longer used.