btashton commented on a change in pull request #1647:
URL: https://github.com/apache/incubator-nuttx/pull/1647#discussion_r476746549
##########
File path: Documentation/Pipfile
##########
@@ -6,10 +6,9 @@ verify_ssl = true
[dev-packages]
[packages]
-recommonmark = "*"
-m2r2 = "==0.2.5"
-Sphinx = "==3.2.1"
-sphinx-rtd-theme = "==0.5.0"
Review comment:
We should be supplying minimum working versions for these. I'm sure
there are versions that will not work.
I would rather just tell people using pipenv to just use pipenv install -r
requirements.txt instead of maintaining things in two places. But I understand
you want a fully pinned requirement list.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
