Jared Dillard

Results 59 comments of Jared Dillard

@lextm Just curious, would using the [dirhtml](https://www.sphinx-doc.org/en/master/usage/builders/index.html) builder work in your case? It changes the build structure to remove the need for `.html` and this extension already supports the `dirhtml`...

I tested out this branch and it does a great job of skipping **roles**, but it seems to remove unknown **directives**. Example: ```diff -Current contributors --------------------- - -Thanks to all...

Do the `replace`, `remove`, `convert`, and `edit` action types currently work with VSCode, they just aren't documented? `suggest` doesn't seem to be supported in VSCode, as the [README](https://github.com/errata-ai/vale-vscode) says "Spelling...

I would argue the original issue hasn't been fixed, as the following things are still confusing: 1. It calls the extension `vale-server`, but from a normal user's point of view,...

My point was to not refer to the marketplace ID in the second install step of the README file (I'm pretty sure as a user I don't ever need to...

I'm not positive, but it seems the extension only parses `.rst`: https://github.com/sphinx-contrib/emojicodes/blob/ec8f49dcefae36ac36ba7100f8c6211b1c39819d/sphinxemoji/sphinxemoji.py#L55 where as it should support everything in [source_suffix](https://www.sphinx-doc.org/en/master/usage/configuration.html#confval-source_suffix)?

myst-parser automatically adds `.md` to `source_suffx`: https://github.com/executablebooks/MyST-Parser/blob/ebf989fdae10c68dfbd3dcdfec4b86259b810cd9/myst_parser/sphinx_ext/main.py#L34 The problem isn't the syntax to be parsed, its the suffix (extension) of the files to be parsed.

Sounds good. I'll try to experiment with setting `parallel_read_safe` to `True` in a fork and report back (only anecdotal evidence, but a start I guess)

Well the testing didn't go well: ``` Exception occurred: File "/path/to/python3.9/site-packages/sphinx/util/parallel.py", line 111, in terminate self._result_funcs.pop(tid) KeyError: 0 ``` I haven't seen that happen before, and this is the closest...

I would. Currently, I have to run them through [swagger2openapi](https://www.npmjs.com/package/swagger2openapi) (not a huge deal), but that would save a step/time/dependency.