-
Notifications
You must be signed in to change notification settings - Fork 118
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
Problem reading logfile causes fatal error #52
Comments
|
Looks like this was a previous issue |
I faced the same issue even specified my own build command
But it still appends the options |
Did you fix this problem? Thx |
For anyone interested, here is the solution I went with. This replaces the broken
|
This solution is inspired by ammaraskar/sphinx-action#52 (comment).
This solution is inspired by ammaraskar/sphinx-action#52 (comment).
When I run this in a github action I get this error:
Looks like the script assumes that there will be a logfile in
/tmp
and when that is not true, then we get an error. I can't actually tell if this is a side-effect ofbuild_all_docs
failing, or if it succeeded, but the log can't be parsed.Perhaps the right fix is to check to make sure the log_file is written before reading it. If it's not written, then inspect the
return_code
and emit a result based on that, together with mention of failure writing the logfile?The text was updated successfully, but these errors were encountered: