`onyo new` default when finding invalid YAML should not be "abort"

[TobiasKadelka]

I created a new asset, somewhere in it there was an accidental tab entered, which the system complained about.
Now our default option is "abort" in case of a yaml error, which I think is terrible, in this case and all others.
It can always happen that somebody enters a format error, or misses a field, or does any other mistake, and the assumption that then the natural way to handle this is "abort the whole thing" is wrong.

I propose to at least change it to "continue (e)diting asset", because as far as I experienced myself and observed in others, the normal situation is that people want to fix a mistake.

Here the situation in which I observed the behavior, command and everything:

(onyo) tkadelka in ~/inventory on git:master
❱ onyo new -t desktop_dell_optiplex_5050 -d <censored> --edit                 1 !
ERROR: Invalid YAML in /var/folders/bt/wtd0rhk11rz_3fb6qq1qhx880000gp/T/onyo_5c175i2x.yaml:
while scanning for the next token
found character '\t' that cannot start any token
  in "/var/folders/bt/wtd0rhk11rz_3fb6qq1qhx880000gp/T/onyo_5c175i2x.yaml", line 16, column 6
Continue (e)diting asset, (s)kip asset or (a)bort command)? [Default: a] e

[aqw]

Yeah... this is a good point. However, this exposes our problem with the --yes switch.

Currently --yes actually means "accept the default". Which if that means to edit, it would be caught in an infinite loop.

So that's the reason why it is currently as it is.

#497 is the work that needs to be done before we can talk about changing defaults of various prompts.