Gh v2 580b czarte #6
Open
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…tor, persistent_configuration_mock
…string_u64_value_pairs
…_command to masq, implemented From(String, usize) for CountryCodes
* continue of code review changes * test for panic in handle_retrieve_transactions * fixing more comments from review * removing TODO * implementing review, added two new tests for ip_country * removing unnecesary country_code from node_descriptor * added cfg(test) and cfg(not(test)) for static ref COUNTRY_CODE_FINDER, created test_dbip_country.rs * change country_code member of NodeRecordInner_0v1 to country_code_opt * simplifying tests in node_record.rs and neighborhood_database.rs * resolving review comments * final changes in gossip_acceptor from review * continue of implementing review comments * fix test return value * added ip_country to ci/all.sh * cherry-pick from GH-784-fix-for-Vojta and implementing iterator for deserializer * try to create test for fmt * tests for print ip v4 and v6 vec[u8] in country_block_serde.rs * ci for ip_country + resolved conflicts of clippy * masq_lib dependencies solved * dependency masq_lib for ip_country in node workspace * remove commented out code + fix condition for non empty bit_queue * formatting * 5 seconds for wait_for_log in requested_chain_meets_different_db_chain_and_panics_integration * fixing impl for AGR in multinode tests * formatting * fix nonexistent NodeAddr in multinode tests for AGR * change single match to if * try to fix build in actions for MacOS * try to find out disk space on MacOS container in Actions * add diagnostics commands on failure * check diskspace before ci/all.sh workflow runs * diagnose mounted disks * analyze /var/ diskspace * remove advanced diagnosticst for disk space * switch macos to version 13 * diagnostics for /Users/runner/work/Node/Node/node/target/release * fasten diagnostics * fasten diagnostics fix * fasten diagnostics back * GH-784-review-two-state: fixes that could work on Actions, appeasing it * fixing review * implementing CountryBlock count in generated rs file, and infrastructure * fixing ip_country tests * fix import of ip_country to multinode_integration_tests * formatting * fix the reference for COUNTRY_CODE_FINDER * implemenitng initalization checker for COUNTRY_CODE_FINDER * CountryCodeFinder initializer * fixed comments from review * removed custom PartialEq for NodeLocation * get back with COUNTRY_CODE_FINDER --------- Co-authored-by: Bert <[email protected]>
ommand for exit location, add test for neighborhood actor to test recieved message
…xit_location_opt, cleanup persistent configuration, test finished for set_exit_location message handler
…ocation_can_be_changed_using_exit_location_msg
…names in generate and commit
…eparate branch in ci-matric workflow
…epting_connections_is_rejected for no reason, wierd behaviour of qualifies
… checking out full version from git, same in actions, remove PARENT_DIR variable for Jenkins
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.