Closed maxandersen closed 3 years ago
This is quite a good thing. In future we can have this thing integrated with the browser extension as well. (i.e The user can chose to open with the existing configurations in config file or can chose a one time configuration to launch starfix)
So now if we dont have the config file(when the user runs starfix first time), he'll get something like this
java.lang.IllegalArgumentException: Clone path not specifed, please specify via --clone-path or use config to set it up.
along with other stacktrace like :
at dev.starfix.Starfix_Subclass.cloneCmd$$superforward1(Starfix_Subclass.zig:142)
at dev.starfix.Starfix_Subclass$$function$$4.apply(Starfix_Subclass$$function$$4.zig:33)
at io.quarkus.arc.impl.AroundInvokeInvocationContext.proceed(AroundInvokeInvocationContext.java:54)
What I was thinking is that shouldn't it display picocli help instead ?
new CommandLine(new Starfix()).usage(System.out); // Will invoke Picocli Help
I got tired of having to edit the config to try things out so changed config reading to be default provider for picocli so config will provide defaults but you can override with command line arguments.