ansible / ansible-content-parser

Apache License 2.0
3 stars 4 forks source link

Change the behavior of --no-exclude option #18

Closed TamiTakamiya closed 1 year ago

TamiTakamiya commented 1 year ago

For changing the behavior of --no-exclude option.

On the current version, it is implemented as:

  --no-exclude          Do not rerun ansible-lint with excluding files that
                        caused syntax check errors. If one or more syntax check
                        errors were found, execution fails without generating
                        the training dataset.

We will change this behavior (with keeping the name of the option) to

  --no-exclude          Do not let ansible-content-parser to generate 
                        training dataset by excluding files that caused lint errors. 
                        With this option specified, a single lint error terminates  
                        the execution without generating the training dataset.

i.e., we will treat syntax check errors and other lint errors in the same way after the change.