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

IBC Hooks #150

Merged
merged 23 commits into from
Sep 11, 2023
Merged

IBC Hooks #150

merged 23 commits into from
Sep 11, 2023

Conversation

faddat
Copy link
Collaborator

@faddat faddat commented Apr 25, 2023

Description and Motivation

This is a blank PR to merge in IBC hooks for Osmosis. The hooks aren't integrated yet.

The goal of this PR is to make Osmosis easy to use for contract authors on Migaloo, and in the end, we're hoping vice versa, too.

Related Issues


Checklist:

  • I have read Migaloo's contribution guidelines.
  • My pull request has a sound title and description (not something vague like Update index.md)
  • All existing and new tests are passing.
  • I updated/added relevant documentation.
  • The code is formatted properly golangci-lint run ./... --fix.

@@ -0,0 +1,185 @@
# IBC-hooks

Check notice

Code scanning / Remark-lint (reported by Codacy)

Warn when a line feed at the end of a file is missing.

[final-newline] Missing newline character at end of file
Copy link

@github-advanced-security github-advanced-security bot left a comment

Choose a reason for hiding this comment

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

Markdownlint (reported by Codacy) found more than 10 potential problems in the proposed changes. Check the Files changed tab for more details.

Copy link

@github-advanced-security github-advanced-security bot left a comment

Choose a reason for hiding this comment

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

Revive (reported by Codacy) found more than 10 potential problems in the proposed changes. Check the Files changed tab for more details.

@faddat faddat mentioned this pull request May 27, 2023
@anhductn2001
Copy link
Contributor

Is it done yet? Do you need any help in this PR? @faddat

@expertdicer expertdicer mentioned this pull request Jun 28, 2023
5 tasks
@faddat faddat changed the base branch from release/v3.0.x to release/v4.0.x September 8, 2023 03:32
@faddat faddat marked this pull request as ready for review September 8, 2023 03:36
@faddat faddat merged commit 4863a20 into release/v4.0.x Sep 11, 2023
19 checks passed
@hoank101 hoank101 deleted the ibc-hooks branch March 22, 2024 01:53
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.

3 participants