fix: test all file

[thomasBousselin]

The previous pipeline only tested the last file

[thomasBousselin]

@xamcost Forgot the commits name it was just supposed to be a test at the beginning.

[xamcost]

@thomasBousselin My bad, my implementation was indeed not exiting with an error if another file than the last one was invalid, sorry about that! Your implementation correct that! However, it stops the loop at the first encountered error, which means that other file would not be checked once one is invalid. My original intention was to check all files and exit with an error if any error is encountered in the loop, but I was wrong. Another way to actually do this would be to build on your solution but exit only with a final check, like this:

error=0
for file in $(find . -name "*.jsonld"); do
  printf "Checking $file .. "
  jq empty $file && echo "OK" || error=1
done

exit $error

That way, all files are checked, even if the first one is invalid, and the code exits with an error if at least one error got raised in the loop. What do you think ?

[thomasBousselin]

Yes it is better. I have added an echo in case of error for readability.

[xamcost]

LGTM! Thank you for this work @thomasBousselin , and for the thorough testing! Have a nice week end ✨

[thomasBousselin]

You to