search

stan-dev / httpstan

HTTP interface to Stan, a package for Bayesian inference.
ISC License
39 stars 15 forks source link

docs: pin jsonschema #644

Closed ahartikainen closed 9 months ago

ahartikainen commented 9 months ago

Pin jsonschema to 4.17.3 to be compatible with sphinx.

ahartikainen commented 9 months ago

We probably want to leave a comment there why are pinning jsonschema

# compatibility with sphinx 4.2

But once we update sphinx we can remove the whole line. Should add a comment about it too?

riddell-stan commented 9 months ago

Have a look at https://github.com/stan-dev/httpstan/pull/642 -- I think I solved the issue with building docs. The compatibility issue involves, I think, sphinxcontrib-openapi and jsonschema. I don't think sphinx itself is implicated.

ahartikainen commented 9 months ago

Good catch

ahartikainen commented 9 months ago

Fix in #642