Skip to content

Add required ReadTheDocs configuration - #294

Open
clenk wants to merge 3 commits into
oasis-open:masterfrom
clenk:readthedocs-conf
Open

Add required ReadTheDocs configuration#294
clenk wants to merge 3 commits into
oasis-open:masterfrom
clenk:readthedocs-conf

Conversation

@clenk

@clenk clenk commented Apr 5, 2024

Copy link
Copy Markdown
Contributor

Should fix broken RTD doc builds. Needed because of https://blog.readthedocs.com/migrate-configuration-v2/

@rpiazza

@clenk

clenk commented Apr 5, 2024

Copy link
Copy Markdown
Contributor Author

Failing due to

AttributeError: module 'lxml.etree' has no attribute '_ElementStringResult'

@clenk

clenk commented Apr 5, 2024

Copy link
Copy Markdown
Contributor Author

More info on failure:
stixmarx uses that etree property here: https://github.com/mitre/stixmarx/blob/4f906eb15af431c12c36c8683f9bcd60d39c364e/stixmarx/xml.py#L59
But it was removed in this commit: lxml/lxml@eba7934

We could pin lxml to <5.1.1 here or fix that line in stixmarx and make a new release.

@CLAassistant

CLAassistant commented Jul 31, 2025

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants