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: use raw types #550

Merged
merged 78 commits into from
Aug 7, 2023
Merged

fix: use raw types #550

merged 78 commits into from
Aug 7, 2023

Conversation

ibrizsabin
Copy link
Collaborator

@ibrizsabin ibrizsabin commented Jul 25, 2023

Description:

Commit Message

fix: use raw types

see the guidelines for commit messages.

Changelog Entry

## [0.1.0] - 2023-03-08
 
 ### Changed

- Removed ibc-rs message types 
- Removed conversions between raw types and ibc-types
- Removed unused code, tests and files.

Checklist:

  • I have performed a self-review of my own code
  • I have documented my code in accordance with the documentation guidelines
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • I have run the unit tests
  • I only have one commit (if not, squash them into one commit).
  • I have a descriptive commit message that adheres to the commit message guidelines

Please review the CONTRIBUTING.md file for detailed contributing guidelines.

@ibrizsabin ibrizsabin marked this pull request as ready for review August 3, 2023 07:10
@ibrizsabin ibrizsabin requested a review from redlarva as a code owner August 3, 2023 07:10
izyak
izyak previously approved these changes Aug 4, 2023
Copy link
Collaborator

@redlarva redlarva left a comment

Choose a reason for hiding this comment

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

looks good

@redlarva redlarva merged commit f168260 into main Aug 7, 2023
@redlarva redlarva deleted the fix/use-raw-types branch August 7, 2023 08:40
@ibrizsabin ibrizsabin linked an issue Aug 8, 2023 that may be closed by this pull request
@pragyanshrestha-ibriz pragyanshrestha-ibriz linked an issue Aug 10, 2023 that may be closed by this pull request
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
6 participants