#150: Replace 11-parameter config_create() with simple config_create() that initializes to sensible defaults. Callers set fields directly instead of passing 11 positional arguments.
#149: Extract validate_config() from main() — reduces main() from 325 to 281 lines. All 8 sequential flag conflict checks moved into a single validate_config() function.
#151: Extract run_dry_run(), connect_to_server(), send_manifest(), and print_progress() shared helpers to eliminate code duplication between send_files() and send_files_multithreaded().
All 22 unit tests pass.
Fixes #149, #150, #151
#150: Replace 11-parameter config_create() with simple config_create() that initializes to sensible defaults. Callers set fields directly instead of passing 11 positional arguments.
#149: Extract validate_config() from main() — reduces main() from 325 to 281 lines. All 8 sequential flag conflict checks moved into a single validate_config() function.
#151: Extract run_dry_run(), connect_to_server(), send_manifest(), and print_progress() shared helpers to eliminate code duplication between send_files() and send_files_multithreaded().
All 22 unit tests pass.
- #150: Replace 11-parameter config_create() with config_create() that
initializes to sensible defaults; callers set fields directly
- #149: Extract validate_config() from main(); reduce main() from 325 to
281 lines by extracting validation logic into separate function
- #151: Extract run_dry_run(), connect_to_server(), send_manifest(), and
print_progress() shared helpers from send_files()/send_files_multithreaded()
to eliminate code duplication
Blocking a user prevents them from interacting with repositories, such as opening or commenting on pull requests or issues. Learn more about blocking a user.
Fixes #149, #150, #151
#150: Replace 11-parameter config_create() with simple config_create() that initializes to sensible defaults. Callers set fields directly instead of passing 11 positional arguments.
#149: Extract validate_config() from main() — reduces main() from 325 to 281 lines. All 8 sequential flag conflict checks moved into a single validate_config() function.
#151: Extract run_dry_run(), connect_to_server(), send_manifest(), and print_progress() shared helpers to eliminate code duplication between send_files() and send_files_multithreaded().
All 22 unit tests pass.
Pull request closed