Skip to content
This repository has been archived by the owner on Apr 22, 2024. It is now read-only.

Fix: cast spell script #247

Merged
merged 1 commit into from
Feb 15, 2024
Merged

Fix: cast spell script #247

merged 1 commit into from
Feb 15, 2024

Conversation

amusingaxl
Copy link
Collaborator

The script is broken since Foundry introduced an additional output when specifying the return type of the function in the signature:

cast call "$MCD_GOV" 'balanceOf(address)(uint)' "$ETH_FROM"

Now returns:

100000000000000000000000 [1e23]

Notice the dangling [1e23].

Previously, it would return simply:

100000000000000000000000

I'm not entirely sure when this change was introduced, but it must be a recent one, since last time I cast a Goerli spell (late 2023), it worked without any issues.

@amusingaxl amusingaxl requested a review from oddaf February 6, 2024 22:40
@amusingaxl
Copy link
Collaborator Author

@amusingaxl amusingaxl marked this pull request as ready for review February 13, 2024 14:25
@amusingaxl amusingaxl merged commit fe9c0a0 into master Feb 15, 2024
3 checks passed
@amusingaxl amusingaxl deleted the fix/cast-spell-script branch February 15, 2024 12:33
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants