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

feat: update ordhook schemas #458

Merged
merged 12 commits into from
Nov 17, 2023
Merged

feat: update ordhook schemas #458

merged 12 commits into from
Nov 17, 2023

Conversation

lgalabru
Copy link
Contributor

@lgalabru lgalabru commented Nov 8, 2023

No description provided.

Copy link
Contributor

@hugocaillard hugocaillard left a comment

Choose a reason for hiding this comment

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

Suggested a refactor that could provide a small readability improvement

components/chainhook-sdk/src/observer/mod.rs Outdated Show resolved Hide resolved
components/chainhook-sdk/src/observer/mod.rs Outdated Show resolved Hide resolved
Ludo Galabru added 7 commits November 16, 2023 14:48
commit b47713c
Merge: b30c843 164c106
Author: Ludo Galabru <[email protected]>
Date:   Thu Nov 16 14:53:00 2023 -0500

    Merge pull request #459 from hirosystems/chore/upgrade-dependencies

    chore: ugprade dependencies from clarinet

commit 164c106
Author: Hugo Caillard <[email protected]>
Date:   Wed Nov 15 17:30:51 2023 +0100

    refactor: review

commit 90dffe1
Author: Hugo Caillard <[email protected]>
Date:   Wed Nov 15 16:29:35 2023 +0100

    chore: ugprade dependencies from clarinet
Copy link
Contributor

@hugocaillard hugocaillard left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@lgalabru lgalabru merged commit 2973c22 into develop Nov 17, 2023
3 of 5 checks passed
@lgalabru lgalabru deleted the feat/ordhook-schemas branch November 17, 2023 03:36
Copy link

🎉 This PR is included in version 1.2.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants