Skip to content

Format #215

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
Merged
merged 8 commits into from
Jul 29, 2022
Merged

Format #215

merged 8 commits into from
Jul 29, 2022

Conversation

@guibescos
Copy link
Contributor

@guibescos guibescos commented Jul 28, 2022 •
edited
Loading

This PR:

  • Add rust formatting to CI. Scope limited to format/
  • Adds a pre-commit hook similar to pyth-sdk-rs to format. The scope is limited to program/. Check README to see how to use.
  • Adds a toplevel Cargo.toml for better developer experience.

@guibescos guibescos marked this pull request as ready for review July 28, 2022 20:29
@guibescos guibescos requested a review from majabbour July 28, 2022 20:30
Guillermo Bescos and others added 3 commits July 28, 2022 20:52
Fix build
3a46596
@guibescos
Log time and ema (#212)
f0bb36c 
* Log time and ema

* Fix ema deserialize

* Revert format

* Add expo

* Fix comments

* Unused import
@guibescos
Revert "Log time and ema (#212)" (#216)
056eb3a 
This reverts commit f0bb36c.
@majabbour
Copy link
Contributor

majabbour commented Jul 29, 2022

LGTM

@guibescos guibescos merged commit f31a788 into main Jul 29, 2022
@guibescos guibescos deleted the rust-format branch July 29, 2022 17:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@majabbour majabbour majabbour approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@guibescos @majabbour