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.
|
🤖 I have created a release *beep* *boop* --- ## [0.5.0](v0.4.0...v0.5.0) (2026-05-29) ### ⚠ BREAKING CHANGES * Deletes the ApproveToken helper and related tests. * Removes the Solana MCMS state views from the public API. * Removes the LinkToken and StaticLinkToken state views from the public API. * Moves version constants out of the public API and into internal. * Remove `ValidateSelectorsInEnvironment` method from `pkg/cldfutil/selectors.go`. ### Features * delete ApproveToken helper ([#70](#70)) ([83f59e7](83f59e7)) * delete Link state views ([#68](#68)) ([d5f643b](d5f643b)) * link refactor ([#71](#71)) ([177dbc5](177dbc5)) * remove Solana MCMS state views ([#69](#69)) ([eb948dd](eb948dd)) * remove ValidateSelectorsInEnvironment method ([#61](#61)) ([72f431f](72f431f)) ### Bug Fixes * move contract version to internal package ([#64](#64)) ([936a00b](936a00b)) --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please). Co-authored-by: app-token-issuer-ops-platform[bot] <275822481+app-token-issuer-ops-platform[bot]@users.noreply.github.com>
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.