Parallel reading warning

[benjdj6]

When running sphinx-build I get the error below:

Warning, treated as error:
the sphinx_markdown_tables extension does not declare if it is safe for parallel reading, assuming it isn't - please ask the extension author to check and make it explicit
✖  sphinx-build (rtfm) failed

Can you mark whether this extension is safe for parallel reading? Here's a similar issue for a different extension w/ an example of marking: https://github.com/agronholm/sphinx-autodoc-typehints/issues/9

[ryanfox]

Thanks for the PR. Is this a necessary change? Per the sphinx docs, it defaults to False:

'parallel_read_safe': a boolean that specifies if parallel reading of source files can be used when the extension is loaded. It defaults to False, i.e. you have to explicitly specify your extension to be parallel-read-safe after checking that it is.

[kapilt]

it is needed, because unless its explicitly marked, sphinx will generate a warning for being implicit. for projects treating warnings as errors (ie ci doc builds), this change is needed.

[kapilt]

also afaict this extension would be parallel safe? its not dealing with any global state

[kapilt]

WARNING: the sphinx_markdown_tables extension does not declare if it is safe for parallel reading, assuming it isn't - please ask the extension author to check and make it explicit                                                                                                                                         
WARNING: doing serial read

[benjdj6]

Thanks @kapilt for the review and participation. I've updated the PR according to your comment.

@ryanfox due to the lack of explicit declaration this is currently broken when running builds in my CI solution. It'd be great if we could get this merged in.