Separate the dev and docs requirements (#708)

* Separate the `dev` and `docs` requirements

Running into trouble installing the custom sphinx packages from `git` in CPU CI, so this would help us get the `dev` dependencies without trying to install `docs` dependencies we don't need in that environment.

* Remove building the docs from the Tensorflow tests

This appears to already be covered by the Github Actions for the docs preview and scheduled rebuild, but is causing issues running the tests for this repo in other Merlin libraries (e.g. `core`.)

.github/workflows/tensorflow.yml

@@ -55,17 +55,6 @@ jobs:
         with:
           name: dist
           path: dist
-      # Build the documentation, TODO: SPHINXOPTS="-W -q"
-      - name: Make HTML
-        run: |
-          make -C docs html
-      - name: Upload HTML
-        uses: actions/upload-artifact@v2
-        with:
-          name: html-build-artifact
-          path: docs/build/html
-          if-no-files-found: error
-          retention-days: 1
       - name: Store PR information
         run: |
           mkdir ./pr

requirements/dev.txt

@@ -6,20 +6,11 @@ flake8
 interrogate==1.5.0
 isort
 mypy
-Sphinx==3.5.4
-jinja2<3.1
 jupyter
 markupsafe==2.0.1
-sphinx_rtd_theme
 sklearn
-natsort==8.1.0
 tensorflow-metadata
 tensorflow-ranking>=0.4
 testbook
-sphinx-external-toc==0.2.4
-myst-nb==0.13.2
-linkify-it-py==1.0.3
-sphinx-multiversion@git+https://github.com/mikemckiernan/sphinx-multiversion.git
-sphinxcontrib-copydirs@git+https://github.com/mikemckiernan/sphinxcontrib-copydirs.git
 fiddle
 wandb

requirements/docs.txt

@@ -0,0 +1,11 @@
+attrs==21.4.0
+Jinja2<3.1
+linkify-it-py==1.0.3
+myst-nb==0.13.2
+natsort==8.1.0
+recommonmark==0.7.1
+sphinx_rtd_theme==1.0.0
+sphinx-external-toc==0.2.4
+sphinx-multiversion@git+https://github.com/mikemckiernan/sphinx-multiversion.git
+Sphinx==3.5.4
+sphinxcontrib-copydirs@git+https://github.com/mikemckiernan/sphinxcontrib-copydirs.git

setup.py

@@ -39,7 +39,9 @@ def read_requirements(filename):
 
 
 _dev = read_requirements("requirements/dev.txt")
+_docs = read_requirements("requirements/docs.txt")
 _nvt = read_requirements("requirements/nvtabular.txt")
+
 requirements = {
     "base": read_requirements("requirements/base.txt"),
     "tensorflow": read_requirements("requirements/tensorflow.txt"),
@@ -49,6 +51,7 @@ def read_requirements(filename):
     "xgboost": read_requirements("requirements/xgboost.txt"),
     "nvtabular": _nvt,
     "dev": _dev,
+    "docs": _docs,
 }
 dev_requirements = {
     "tensorflow-dev": requirements["tensorflow"] + _dev + _nvt,