Fix python requirements #8034
Fix python requirements #8034
Conversation
There was a problem hiding this comment.
Hi @esnyder-rve
Fixes issue with docutils dependency on fresh builds
Afaics, this is no more the case, isn't it? A fresh build takes proper dependencies and I couldn't spot any regression regarding our concerns.
This also makes the python environment match the website.
I very much like getting latest versions and play with them but considering the recent report, I must admit that I couldn't find any reasonable arguments against your point (I secretly expected someone else would do but... 🥲 ). Let's however be minimalistic and not fetch everything from the website.
re transifex-client and sphinx[transifex], these are deprecated dependencies we should get rid of (they even result in warnings in build reports). QGIS-Documentation does not need them at the server level given that all the translation processes (config/pull/push/update) are currently done between Transifex and GitHub directly; the server "just" builds from strings in the repo. But we need to upgrade to the tx-cli before removal (I suspect our push action will fail next Tuesday)
Fix underway.
| # Memo: Should also be added to https://github.com/qgis/QGIS-Website/blob/master/REQUIREMENTS.txt | ||
| # for docker image builds |
There was a problem hiding this comment.
Can we keep these lines, please? They are meant to help remember fixing the server dependencies when we add new ones to the docs
| sphinx-intl[transifex] | ||
| icalendar |
There was a problem hiding this comment.
We can remove this one as we don't need/use it here
| icalendar |
| sphinxext-rediraffe | ||
| sphinx_togglebutton | ||
| requests==2.25.1 |
There was a problem hiding this comment.
ditto
| requests==2.25.1 |
|
@esnyder-rve #8065 removed transifex dependency |
|
superseded by #8110 |
Goal:
Fixes issue with
docutilsdependency on fresh builds. This also makes the python environment matchthe website. I just pulled the
REQUIREMENTS.txtfrom the QGIS-Website repo and added back the missingtransifix-clientthat is missing from the website version and therefore was removed.Ticket(s): #
fix #8024
Notes:
Some of the "deletions" are really just items being re-ordered. As stated above, the
transifix-clientitem was the only one actually removed (and put back).