Currently, when PyYAML is not installed, rqstats -y prints an error message, but it still exits with a 0 exit code, indicating success. Fix that, along with a few related issues:
The error message was printed to stdout rather than stderr.
The name of the library in the message was incorrect (LibYAML is a C library; the Python library is PyYAML).
After this change, you need to install PyYAML to run the testsuite, so modify the GitHub workflow accordingly. Also, make sure to install all packages at once, to make sure that later install commands don't overwrite the result of the earlier ones.
selwin
commented
2 months ago
Currently, when PyYAML is not installed,
rqstats -yprints an error message, but it still exits with a 0 exit code, indicating success. Fix that, along with a few related issues:The error message was printed to stdout rather than stderr.
The name of the library in the message was incorrect (LibYAML is a C library; the Python library is PyYAML).
After this change, you need to install PyYAML to run the testsuite, so modify the GitHub workflow accordingly. Also, make sure to install all packages at once, to make sure that later install commands don't overwrite the result of the earlier ones.