Skip to content

Conversation

@Eric-Warehime
Copy link
Contributor

Description

Bumps the ibc-go dependency to point to the most recent commit on main.


Author Checklist

All items are required. Please add a note to the item if the item is not applicable and
please add links to any relevant follow up issues.

I have...

  • tackled an existing issue or discussed with a team member
  • left instructions on how to review the changes
  • targeted the main branch

@Eric-Warehime Eric-Warehime requested a review from srdtrk October 24, 2025 16:24
@Eric-Warehime
Copy link
Contributor Author

@srdtrk Still working on resolving everything here, but I think I've wired the new IBC changes incorrectly. Let me know if there is anything that stands out to you.

@Eric-Warehime
Copy link
Contributor Author

Fixed everything up.

@Eric-Warehime Eric-Warehime changed the title chore: Bump ibc-go chore!: Bump ibc-go Oct 27, 2025
@aljo242 aljo242 enabled auto-merge October 27, 2025 19:33
Comment on lines +249 to +251
for _, k := range keys {
nonTransientKeys = append(nonTransientKeys, k)
}

Check warning

Code scanning / CodeQL

Iteration over map Warning

Iteration over map may be a possible source of non-determinism
@aljo242 aljo242 added this pull request to the merge queue Oct 28, 2025
Merged via the queue into main with commit de4f0d2 Oct 28, 2025
19 of 20 checks passed
@aljo242 aljo242 deleted the eric/bump-ibc-go branch October 28, 2025 15:55
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.

4 participants