hpi-swa / smalltalkCI

Framework for testing Smalltalk projects with GitHub Actions, GitLab CI, Travis CI, AppVeyor, and others.
MIT License
96 stars 68 forks source link

CLI: Fix --clean option if configuration is absent #520

Closed LinqLover closed 3 years ago

LinqLover commented 3 years ago

run.sh: In main(), invert the invocation order of check_clean_up and ensure_ston_config_exists, to prevent error messages regarding the absence of configuration when the command was invoked to clean up the cache only. Resolve the existing dependency order which was manifested by the config_smalltalk variable (read in check_clean_up and written in ensure_ston_config_exists) by moving the "Continue with build progress?" prompt into the latter function, ensure_ston_config_exists. We only show this prompt if the local variable first_action has been set, which at the moment only happens in check_clean_up.

Fixes #519.