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

deps: bump ibc-go to v5 #1361

Merged
merged 3 commits into from
Sep 29, 2022
Merged

deps: bump ibc-go to v5 #1361

merged 3 commits into from
Sep 29, 2022

Conversation

fedekunze
Copy link
Contributor

Closes: #XXX

Description

For contributor use:

  • Targeted PR against correct branch (see CONTRIBUTING.md)
  • Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
  • Code follows the module structure standards.
  • Wrote unit and integration tests
  • Updated relevant documentation (docs/) or specification (x/<module>/spec/)
  • Added relevant godoc comments.
  • Added a relevant changelog entry to the Unreleased section in CHANGELOG.md
  • Re-reviewed Files changed in the Github PR explorer

For admin use:

  • Added appropriate labels to PR (ex. WIP, R4R, docs, etc)
  • Reviewers assigned
  • Squashed all commits, uses message "Merge pull request #XYZ: [title]" (coding standards)

@fedekunze fedekunze marked this pull request as ready for review September 28, 2022 21:38
@fedekunze fedekunze requested a review from a team as a code owner September 28, 2022 21:38
@fedekunze fedekunze requested review from hanchon and ramacarlucho and removed request for a team September 28, 2022 21:38
@fedekunze fedekunze merged commit 5759e72 into main Sep 29, 2022
@fedekunze fedekunze deleted the fedekunze/bump-ibc-go branch September 29, 2022 15:21
@danburck danburck mentioned this pull request Nov 30, 2022
Closed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@hanchon hanchon Awaiting requested review from hanchon hanchon is a code owner automatically assigned from evmos/core-engineering

@ramacarlucho ramacarlucho Awaiting requested review from ramacarlucho ramacarlucho is a code owner automatically assigned from evmos/core-engineering

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
1 participant
@fedekunze