Resume/abort - #4036
Conversation
|
Well holy crap that was fast 😅 |
|
Comes from a fork so this is passage 2. Also my logic was bad your pre-commit checks help a lot |
|
I'll mark this as ready as the logic looks sound and feels correct on testing branch |
|
Thoughts on this one @svartkanin? :) (Aside from having to move it to the new TUI in #3997) |
|
|
||
| def has_saved_config() -> bool: | ||
| """Check if there's a saved config in /var/log/archinstall""" | ||
| config_file = logger.directory / 'user_configuration.json' |
There was a problem hiding this comment.
the file names are defined here
, to make sure any future changes are easy we should define them once and reuse a variable instead of c/p
|
In addition to the comments, I can see some benefits for it but also it's not fully self-contained for regular users as the If the intention with this is to "be easier during dev" I'd suggest to put this behind the |
I agree, skipping credentials is good and if that's the only step when re-running it still save time. And putting it behind |
Integrate resume behind --debug flag - With --debug: "Save selections and abort", "Abort without saving", "Cancel" - Without --debug: "Abort", "Cancel"
|
It's now hidden behind Also tested a little, and is really helpful to be able to test stuff faster. |
Remove mentions of hardcoded paths.
|
This conversation would still be nice to re-open for dev :) |
3 participants
#4035 draft (comes from my dev fork)
This would be especially good for testing ;)