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.

Sign up for GitHub

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

Jump to bottom

fix: struct naming according to docs #55

Merged
merged 2 commits into from
Sep 6, 2023
Merged

fix: struct naming according to docs #55

merged 2 commits into from
Sep 6, 2023

Conversation

gcranju
Copy link
Collaborator

@gcranju gcranju commented Sep 1, 2023

Description:

Commit Message

type: commit message

see the guidelines for commit messages.

Changelog Entry

version: <log entry>

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.

@nightowl121
Copy link
Collaborator

Please fix the failing cargo fmt check

@gcranju gcranju added the bug Something isn't working label Sep 4, 2023
nightowl121
nightowl121 approved these changes Sep 5, 2023
View reviewed changes
Copy link
Collaborator

@nightowl121 nightowl121 left a comment

Choose a reason for hiding this comment

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

Approved

@redlarva redlarva force-pushed the 261-struct-messages-not-according-to-specs branch from 33fd77f to 30ae5ac Compare September 6, 2023 04:07
redlarva
redlarva approved these changes Sep 6, 2023
View reviewed changes
Copy link
Contributor

@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 force-pushed the 261-struct-messages-not-according-to-specs branch from 30ae5ac to 36baf6a Compare September 6, 2023 04:20
@redlarva redlarva merged commit f1f02cc into main Sep 6, 2023
8 checks passed
@redlarva redlarva deleted the 261-struct-messages-not-according-to-specs branch September 6, 2023 04:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@redlarva redlarva redlarva approved these changes

@nightowl121 nightowl121 nightowl121 approved these changes

@ibrizsabin ibrizsabin Awaiting requested review from ibrizsabin ibrizsabin is a code owner

@AntonAndell AntonAndell Awaiting requested review from AntonAndell AntonAndell is a code owner

Assignees
No one assigned
Labels
Audit Issue bug Something isn't working Cosmwasm Rust
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@gcranju @nightowl121 @redlarva