Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ODFValidator & Exit Code #112

Closed
Progi1984 opened this issue Jul 7, 2021 · 4 comments · Fixed by #121
Closed

ODFValidator & Exit Code #112

Progi1984 opened this issue Jul 7, 2021 · 4 comments · Fixed by #121
Milestone

Comments

@Progi1984
Copy link

Hi team members,

I would like to add ODFValidator to PHPPresentation for validating output files.

In my scenario, I would like that Github Actions fails if there are errors in validation.
But actually, if there errors or not the exit code equals 0.

@mistmist
Copy link
Contributor

mistmist commented Jul 8, 2021

hmm the last time i considered this, i wrote:

commit b14e2f6

validator: return a status code indicating success or failure

It is a useful convention to return a failing status code in case
something went horribly wrong during execution, or the command line
arguments are invalid.  This can then be checked e.g. in LO's
test::BootstrapFixture.

(If validation errors are found, it was still a successful execution.)

... but maybe some people want to rely on the exit status to signal validation errors and some are bothered by that so maybe we could have a command line option --fail-on-validation-errors or something like that?

@Progi1984
Copy link
Author

@mistmist It could be perfect. And that would allow to use in CI.

@Progi1984
Copy link
Author

Progi1984 commented Aug 27, 2021

@sergey-s-betke Thanks for the PR.

@mistmist Sorry for disturbing you : I don't find a nightly build version on https://repo1.maven.org/maven2/org/odftoolkit/odfvalidator/. Where can I find it ?

@mistmist
Copy link
Contributor

i dont think the CI we use uploads builds anywhere... you'll have to build from the "master" branch locally until there 0.10 release

@mistmist mistmist added this to the 0.10.0 milestone Nov 29, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants