spdx-3-model icon indicating copy to clipboard operation
spdx-3-model copied to clipboard
verified publisher icon spdx

Print verbose info during PR validation (target: 'develop' branch)

Open bact opened this issue 11 months ago • 5 comments

  • Show the version of the spec-parser currently working with
  • Add --debug --verbose to spec-parser options to print more information for the PR author to understand why the purposed changes failed the validation (if it failed)

bact avatar Jan 23 '25 10:01 bact

What is the purpose of the "validation"? I assume to make sure that the input is "correct" / "valid".

What does this mean? More specifically:

  • errors should most definitely be flagged
  • what about warnings?

And how is it checked?

  • is only the exit code of the process?
  • or is the generated output / error messages also checked?

zvr avatar Jan 23 '25 13:01 zvr

If it's valid it's good.

If it's not, having some information to understand why it is the case is good.

bact avatar Jan 23 '25 13:01 bact

@bact - Can you rebase to trigger the CI? The old CI failed, but it is too stale to see the logs

@zvr - any concerns about merging?

goneall avatar Oct 10 '25 17:10 goneall

Thanks. I will rebase / check the parameters again to keep up with latest spec-parser.

bact avatar Oct 10 '25 18:10 bact

Looks like this is still failing the CI

@bact - can you check?

goneall avatar Nov 14 '25 22:11 goneall