ENH: Compatibility with docutils>=0.18 (from support with sphinx>=5) (#60)

* pre-commit file update

* removing python 3.6

* removing python 3.6

* updated tests

* sphinx version

* sphinx versions

* specific test files for sphinx4

* remove py37 support

* update CI

* removing python3.7

* updating tests for sphinx5

Co-authored-by: mmcky <[email protected]>

Author: AakashGfude

.github/workflows/ci.yml

@@ -13,17 +13,17 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v2
-      - name: Set up Python 3.8
+      - name: Set up Python 3.9
         uses: actions/setup-python@v1
         with:
-          python-version: 3.8
+          python-version: 3.9
       - uses: pre-commit/[email protected]
 
   tests:
     runs-on: ubuntu-latest
     strategy:
       matrix:
-        python-version: [3.6, 3.7, 3.8]
+        python-version: [3.8, 3.9]
     steps:
       - uses: actions/checkout@v2
       - name: Set up Python ${{ matrix.python-version }}
@@ -40,10 +40,10 @@ jobs:
       - name: Create cov
         run: coverage xml
       - name: Upload to Codecov
-        if: matrix.python-version == 3.7
+        if: matrix.python-version == 3.8
         uses: codecov/codecov-action@v1
         with:
-          name: sphinx-exercise-pytest-py3.7
+          name: sphinx-exercise-pytest-py3.8
           flags: pytests
           file: ./coverage.xml
           fail_ci_if_error: true
@@ -53,10 +53,10 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v2
-      - name: Set up Python 3.8
+      - name: Set up Python 3.9
         uses: actions/setup-python@v1
         with:
-          python-version: 3.8
+          python-version: 3.9
       - name: Install dependencies
         run: |
           python -m pip install --upgrade pip
@@ -75,10 +75,10 @@ jobs:
     steps:
       - name: Checkout source
         uses: actions/checkout@v2
-      - name: Set up Python 3.7
+      - name: Set up Python 3.9
         uses: actions/setup-python@v1
         with:
-          python-version: 3.7
+          python-version: 3.9
       - name: Build package
         run: |
           pip install wheel

.pre-commit-config.yaml

@@ -15,19 +15,19 @@ exclude: >
 
 repos:
 
-  - repo: git://github.com/pre-commit/pre-commit-hooks
-    rev: v2.2.3
+  - repo: https://github.com/pre-commit/pre-commit-hooks
+    rev: v4.3.0
     hooks:
       - id: check-yaml
       - id: end-of-file-fixer
       - id: trailing-whitespace
 
-  - repo: https://gitlab.com/pycqa/flake8
-    rev: 3.7.9
+  - repo: https://github.com/PyCQA/flake8
+    rev: 5.0.4
     hooks:
     - id: flake8
 
   - repo: https://github.com/psf/black
-    rev: 20.8b1
+    rev: 22.8.0
     hooks:
       - id: black

docs/requirements.txt

@@ -1,3 +1,3 @@
-sphinx>=3.0
+sphinx>=4,<6
 sphinx-book-theme
 myst-nb

docs/runtime.txt

@@ -1 +1 @@
-3.7
+3.8

setup.py

@@ -18,21 +18,23 @@
 
 # Define all extras
 extras = {
-    "code_style": ["flake8<3.8.0,>=3.7.0", "black", "pre-commit==1.17.0"],
+    "code_style": ["flake8<3.8.0,>=3.7.0", "black", "pre-commit"],
     "testing": [
         "coverage",
         "pytest>=3.6,<4",
         "pytest-cov",
         "pytest-regressions",
         "beautifulsoup4",
-        "myst-nb",
+        "myst-nb~=0.17.1",
+        "sphinx>=4,<6",
+        "docutils>=0.15,<0.19",
         "texsoup",
         "matplotlib",
     ],
     "rtd": [
-        "sphinx>=3.0",
+        "sphinx>=4,<6",
         "sphinx-book-theme",
-        "myst-nb",
+        "myst-nb~=0.17.1",
     ],
 }
 
@@ -41,7 +43,7 @@
 setup(
     name="sphinx-exercise",
     version=VERSION,
-    python_requires=">=3.6",
+    python_requires=">=3.8",
     author="QuantEcon",
     author_email="[email protected]",
     url=BASE_URL,
@@ -55,7 +57,7 @@
     long_description_content_type="text/markdown",
     license="BSD",
     packages=find_packages(),
-    install_requires=["docutils>=0.15", "sphinx", "sphinx-book-theme"],
+    install_requires=["sphinx>=4", "sphinx-book-theme"],
     extras_require=extras,
     include_package_data=True,
     classifiers=[
@@ -67,9 +69,8 @@
         "License :: OSI Approved :: MIT License",
         "Natural Language :: English",
         "Operating System :: OS Independent",
-        "Programming Language :: Python :: 3.6",
-        "Programming Language :: Python :: 3.7",
         "Programming Language :: Python :: 3.8",
+        "Programming Language :: Python :: 3.9",
         "Programming Language :: Python",
         "Topic :: Documentation :: Sphinx",
         "Topic :: Documentation",

sphinx_exercise/__init__.py

@@ -66,7 +66,7 @@
 
 
 def purge_exercises(app: Sphinx, env: BuildEnvironment, docname: str) -> None:
-    """ Purge sphinx_exercise registry """
+    """Purge sphinx_exercise registry"""
 
     if not hasattr(env, "sphinx_exercise_registry"):
         return
@@ -85,7 +85,7 @@ def purge_exercises(app: Sphinx, env: BuildEnvironment, docname: str) -> None:
 def merge_exercises(
     app: Sphinx, env: BuildEnvironment, docnames: Set[str], other: BuildEnvironment
 ) -> None:
-    """ Merge sphinx_exercise_registry """
+    """Merge sphinx_exercise_registry"""
 
     if not hasattr(env, "sphinx_exercise_registry"):
         env.sphinx_exercise_registry = {}
@@ -109,7 +109,7 @@ def init_numfig(app: Sphinx, config: Config) -> None:
 
 
 def copy_asset_files(app: Sphinx, exc: Union[bool, Exception]):
-    """ Copies required assets for formating in HTML """
+    """Copies required assets for formating in HTML"""
 
     static_path = (
         Path(__file__).parent.joinpath("assets", "html", "exercise.css").absolute()

sphinx_exercise/directive.py

@@ -32,7 +32,7 @@
 
 class SphinxExerciseBaseDirective(SphinxDirective):
     def duplicate_labels(self, label):
-        """ Check for duplicate labels """
+        """Check for duplicate labels"""
 
         if not label == "" and label in self.env.sphinx_exercise_registry.keys():
             docpath = self.env.doc2path(self.env.docname)

sphinx_exercise/transforms.py

@@ -28,7 +28,7 @@ class CheckGatedDirectives(SphinxTransform):
     default_priority = 1
 
     def check_structure(self, registry):
-        """ Check Structure of the Gated Registry"""
+        """Check Structure of the Gated Registry"""
         error = False
         docname = self.env.docname
         if docname in registry:
@@ -111,7 +111,7 @@ def apply(self):
                     new_node += child
             # Collect nodes attached to the Parent Node until :solution-end:
             content = docutils.nodes.section(
-                ids="solution-content"
+                ids=["solution-content"]
             )  # TODO: should id be classes?
             for child in parent.children[parent_start + 1 : parent_end]:
                 content += child

tests/conftest.py

@@ -27,14 +27,24 @@ def read(app):
 
 @pytest.fixture
 def get_sphinx_app_doctree(file_regression):
-    def read(app, docname="index", resolve=False, regress=False, flatten_outdir=False):
+    def read(
+        app,
+        docname="index",
+        resolve=False,
+        regress=False,
+        flatten_outdir=False,
+        sphinx_version=False,
+    ):
         if resolve:
             doctree = app.env.get_and_resolve_doctree(docname, app.builder)
             extension = ".resolved.xml"
         else:
             doctree = app.env.get_doctree(docname)
             extension = ".xml"
 
+        if sphinx_version:
+            extension = sphinx_version + extension
+
         # convert absolute filenames
         for node in doctree.traverse(lambda n: "source" in n):
             node["source"] = Path(node["source"]).name

tests/test_build.py

@@ -37,4 +37,4 @@ def test_warnings(app, warnings, wmsg):
     build_path = app.srcdir.joinpath("_build")
     shutil.rmtree(build_path)
     app.build()
-    assert wmsg in warnings(app)
+    assert wmsg in warnings(app).replace("'", "")