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

Gh 468 review3 - with review2 #544

Closed
wants to merge 17 commits into from
Closed

Gh 468 review3 - with review2 #544

wants to merge 17 commits into from

Conversation

czarte
Copy link
Collaborator

@czarte czarte commented Oct 24, 2024

No description provided.

@czarte
fix multinode neighborhood constructor
516712a
@czarte
rename set_exit_location_command.rs to exit_location_command.rs
24ab306
@czarte
add protection for empty Neighborhood DB
f14ab6d
@czarte
change protection for empty Neighborhood
4be9596
@czarte
fixed shared schema, added exit location to --help of masq
9a4c687
@czarte
set version for sccache
eb3ca81
@czarte
add lifetime for daemon_shared_app fn return
aa01112
@czarte
fixing log assertion in tests, add db to exit location test, fix app …
c3e36f2 
…lifetime for daemon help
@czarte
introduce test utilities
129ec37
@czarte
remove unused import - localhost linux compile and passes all ip_coun…
7174c9f 
…try tests
@czarte
addressing comments from review
e21b545
@czarte
addressing comments from review, refactoring Neighborhood for exit-lo…
9208137 
…cation, introduce enum for ExitPreference, fixed Display for ExitLocationSet
@czarte
fix tests for separate_u64_values
0771086
@czarte
add Eq to ExitPreference
60dc65a
@czarte
rearranged test_utilities, fix to exit_location_handler and its tests
de2a90d
@czarte
fixing ByteArrayWriter and Reader for masq
6307d39
@czarte
added more nodes to exit_location test with fallback to test UNREACHA…
34ee03b 
…BLE_COUNTRY_PENALTY
dnwiebe
dnwiebe reviewed Oct 25, 2024
View reviewed changes
ip_country/Cargo.toml Show resolved Hide resolved
masq/Cargo.toml Show resolved Hide resolved
masq/src/commands/exit_location_command.rs Show resolved Hide resolved
masq/src/commands/exit_location_command.rs Show resolved Hide resolved
masq/src/commands/exit_location_command.rs Show resolved Hide resolved
node/src/neighborhood/mod.rs Show resolved Hide resolved
node/src/neighborhood/mod.rs Show resolved Hide resolved
node/src/neighborhood/mod.rs Show resolved Hide resolved
node/src/neighborhood/mod.rs Show resolved Hide resolved
node/src/neighborhood/mod.rs Show resolved Hide resolved
@czarte czarte closed this Nov 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dnwiebe dnwiebe dnwiebe left review comments

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
@czarte @dnwiebe