Skip to content

Conversation

guibescos
Copy link
Contributor

@guibescos guibescos commented Jul 25, 2022

Adds more logging and changes instruction names to Pascal case.

majabbour and others added 30 commits July 21, 2022 21:43
* changed c return value of update price to indicate aggregation

* forgot to add bindings

* removed the need for casting

* fixed update_no_fail

* removed unnecessary docker packages
* changed c return value of update price to indicate aggregation

* forgot to add bindings

* removed the need for casting

* fixed update_no_fail

* removed unnecessary docker packages
@guibescos guibescos marked this pull request as ready for review July 25, 2022 16:11
Copy link
Contributor

@cctdaniel cctdaniel left a comment

Choose a reason for hiding this comment

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

lgtm on the logging messages, thanks for this ser 🤝🏻

@guibescos guibescos merged commit f514be5 into main Jul 25, 2022
@guibescos guibescos deleted the rust-logging-2 branch July 25, 2022 16:33
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