Skip to content

Coax1d digital cash solution #8

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

Open
coax1d wants to merge 11 commits into
base: solutions
Choose a base branch
from
Open

Coax1d digital cash solution #8

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
11 commits
Select commit Hold shift + click to select a range
f6da8d6
adding helpers for exercises and a few initial tests
coax1d Jan 6, 2023
b120dd9
Adding additional tests
coax1d Jan 6, 2023
d2d8db1
Adding test for spending non-existent bill
coax1d Jan 6, 2023
8dce42d
readding todo
coax1d Jan 6, 2023
9527e09
removing irrelevant comment
coax1d Jan 6, 2023
3503780
Removing pub member from state
coax1d Jan 6, 2023
15cdc7b
Merge branch 'main' into coax1d-digital-cash-exercise
JoshOrndorff Jan 9, 2023
207adb7
Adding more tests and correcting incorrect ones
coax1d Jan 12, 2023
b0da136
Merge branch 'coax1d-digital-cash-exercise' of github.com:JoshOrndorf…
coax1d Jan 12, 2023
706a428
Fixes
coax1d Jan 15, 2023
e763499
Solution + more tests
coax1d Jan 15, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion src/c1_state_machine/mod.rs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@ pub trait StateMachine {
}

/// A set of play users for experimenting with the multi-user state machines
#[derive(Hash, Eq, PartialEq, Debug, Clone)]
#[derive(Hash, Eq, PartialEq, Debug, Clone, Ord, PartialOrd)]
Copy link
Owner

@JoshOrndorff JoshOrndorff Jan 15, 2023

Choose a reason for hiding this comment

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

I think we also won't need these traits if we use a HashMap for deduplicating

Copy link
Collaborator Author

@coax1d coax1d Jan 15, 2023

Choose a reason for hiding this comment

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

true

pub enum User {
Alice,
Bob,
Expand Down
Loading