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

Add Rust 1 61 toolchain #65

Merged
merged 4 commits into from
May 20, 2022
Merged

Add Rust 1 61 toolchain #65

merged 4 commits into from
May 20, 2022

Conversation

pefontana
Copy link
Collaborator

@pefontana pefontana commented May 20, 2022 •
edited
Loading

Solves issue #61

  • Add rust toolchain, so that every person that works on the project uses Rust 1.61.0 version
  • Add Rust 1.61.0 and cargo fmt check to the GitHub actions
  • run cargo fmt

@unbalancedparentheses unbalancedparentheses merged commit e92d4a7 into main May 20, 2022
@unbalancedparentheses unbalancedparentheses deleted the add-rust-1-61-toolchain branch May 20, 2022 20:47
azteca1998 pushed a commit that referenced this pull request Feb 10, 2023
@pefontana
Implement StateReader trait for CachedState (#62)
e667b13 
* implement StateCache

* Modify StateReader trait

* Implement StatedReader for CachedStated

* Use reference in funtions

* Add StateCache unit tests

* Rename src/state/state_chache.rs -> src/state/cached_state.rs

* Revert "Rename src/state/state_chache.rs -> src/state/cached_state.rs"

This reverts commit d34a0ea4c64c70dea3f894b6c964e7b1ffcce255.

* Create mod cached_state.rs

* Remove unwraps

* Modify StateReader methods to return references

* Reorder CachedState methods

* Implement State trait for CachedState (#65)

* Add State:Errors

* Implement State for CachedState

* Solve merge conflicts

* cargo fmt

Co-authored-by: Pedro Fontana <pedro.fontana@lamdaclass.com>

Co-authored-by: Pedro Fontana <pedro.fontana@lamdaclass.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@pefontana @unbalancedparentheses