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

fix: Fix ASA-2025-004 #11

Merged
merged 1 commit into from
Feb 28, 2025
Merged

Conversation

JeancarloBarrios
Copy link

@JeancarloBarrios JeancarloBarrios commented Feb 27, 2025

Ptach for ASA-2025-004

@JeancarloBarrios JeancarloBarrios marked this pull request as ready for review February 28, 2025 02:53
@JeancarloBarrios JeancarloBarrios force-pushed the JeancarloBarrios/ibc-encoding-patch branch from 73258b2 to 233ca6a Compare February 28, 2025 02:53
Copy link

@michaelfig michaelfig left a comment

Choose a reason for hiding this comment

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

LGTM! I'll make the two minor import ordering changes.

"fmt"
host "github.com/cosmos/ibc-go/v6/modules/core/24-host"

Choose a reason for hiding this comment

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

I prefer leaving this below porttypes.

Choose a reason for hiding this comment

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

Done!

@@ -1,14 +1,18 @@
package transfer_test

import (
"errors"
sdk "github.com/cosmos/cosmos-sdk/types"

Choose a reason for hiding this comment

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

Please put this above sdkerrors.

Choose a reason for hiding this comment

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

Done!

@michaelfig michaelfig force-pushed the JeancarloBarrios/ibc-encoding-patch branch from 233ca6a to 32d726f Compare February 28, 2025 03:48
@michaelfig michaelfig merged commit e5411dc into Agoric Feb 28, 2025
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