Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix readthedocs build #1719

Closed

Conversation

mjurbanski-reef
Copy link

@mjurbanski-reef mjurbanski-reef commented Aug 20, 2023

fixes regression caused by #1666
you can see in that PR that readthedocs started failing there already

defusedxml==0.7.1
# via jira (setup.cfg)
docutils==0.20.1
docutils==0.19
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

? this isnt right it seems

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

but you gave me some ideas and i fixed hte build ... could be indeed some deps are to be deleted though

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

see #1669

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thank you for you contribution. reopen and fix if you think your PR might still have some value (like deleting some deps from constraints.txt

@studioj studioj closed this Sep 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants