Skip to content

Commit

Permalink
docs: Drop wavedrom, reorder requirements
Browse files Browse the repository at this point in the history
Sphinx-wavedrom, cairocffi, CairoSVG don't seem to be tested
continuously, which leads to fatal bugs reaching production.
The extension in particular has two design oversights:
* It renders on the client side, requiring wavedrom.js to be
  fetched from a different from a different domain, slowing
  down or even preventing the documentation if the domain is
  offline (cause of a previous problem).
* Even though it renders on the client side in our user case, it
  still cairo renderer during build, even though it is unused (the
  root cause of the current issue)

Drop it in favor of just committing the wavedrom source (commented out
in the page or as a text file in the same path) and the rendered svg.

Also, reorder requirements to have adi_doctools as the first
requirement.

Signed-off-by: Jorge Marques <[email protected]>
  • Loading branch information
gastmaier committed Jan 13, 2025
1 parent fdda4ef commit e0dfeb0
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 3 deletions.
1 change: 0 additions & 1 deletion docs/conf.py
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,6 @@

extensions = [
"sphinx.ext.todo",
"sphinxcontrib.wavedrom",
"adi_doctools"
]

Expand Down
3 changes: 1 addition & 2 deletions docs/requirements.txt
Original file line number Diff line number Diff line change
@@ -1,3 +1,2 @@
sphinx
https://github.com/analogdevicesinc/doctools/releases/download/latest/adi-doctools.tar.gz
sphinxcontrib.wavedrom
sphinx

0 comments on commit e0dfeb0

Please sign in to comment.