search

usnistgov / Docs4NIST

ReadTheDocs work-alike for Sphinx documentation at the US National Institute of Standards and Technology
https://pages.nist.gov/Docs4NIST
Other
2 stars 3 forks source link

pre-build step not working correctly #17

Open guyer opened 1 year ago

guyer commented 1 year ago

As reported by @ianhbell via email: "There is something wrong in @main with pre-build. The pre-build seems to be interpreted in a different way than before, and it doesn’t find python to run my pre-build script. I thought maybe it was the multi-line pre-build script you helped me set up before, so I tried to pull it all into a single python script, but that doesn’t fix it either, it can’t find python. Perhaps it is how subprocess is being used to call the pre-build command. See: https://github.com/ianhbell/teqp-docs/actions/runs/6331159254/job/17195114598"

This happened after intentionally removing shell=True. Maybe interpretation of multi-line YAML strings is different?

ianhbell commented 1 year ago

I think this was exactly as you wrote in the email about the appending of |- to the end of the field in YML

guyer commented 1 year ago

OK. That's a sign that the documentation needs to be really explicit.

ianhbell commented 1 year ago

I am currently testing shell=False in my fork with the argument as you suggested

ianhbell commented 1 year ago

shell=False is still not working for me with multi-line scripts

guyer commented 1 year ago

OK, I'll try to figure out what's going on