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

Adopt sphinx design #5127

Merged
merged 25 commits into from
Apr 18, 2023
Merged

Adopt sphinx design #5127

merged 25 commits into from
Apr 18, 2023

Conversation

tkknight
Copy link
Contributor

@tkknight tkknight commented Jan 4, 2023

🚀 Pull Request

Description

sphinx-panels was being used in the Iris docs for the cards, pulldowns and some icons.

sphinx-panels is now not maintained, instead the developer has moved to sphinx-design.

This PR migrates the Iris docs from sphinx-panels to sphinx-design.

Changes:

  • Indentation fixes (bullet points and images not aligning correctly)
  • Adopted sphinx-design (retired sphinx-panels)
    • dropdown directive syntax changed

    • updated past whatsnew pages that uses dropdown to use coloured icons

      • old
        image
      • new
        image
    • sphinx v5.3 is now used as it was previously pinned to v4.5 by sphinx-panels.

      • pinned to v5.3 for now as readthedocs jumps to v6.1.3 even though when a conda env is build locally it only uses v5.3. Also the v6.1.3 build docs does not seem to render the Voted Issue List table at all - for now lets pin at v5.3 and review later.

      • sphinx now seems to create the bookmarks on the API pages correctly, so you can jump to functions using the sidebar, nice.

        • old
          image
        • new
          image
      • Removed sphinxext/custom_class_autodoc.py as it is no longer needed (and sphinx complains about it now)

      • Used newer syntax for extlinks on conf.py

Closes #5121


Consult Iris pull request check list

@tkknight
Copy link
Contributor Author

tkknight commented Jan 4, 2023

A fix (executablebooks/sphinx-design#109) is needed for sphinx-design to make the presentation of the cards on the Iris docs homepage. Let's sees if it can be fixed before Iris 3.5 due ~April 2023.

The alternative is some custom css or making all icon images pngs which I would like to avoid.

* upstream/main: (59 commits)
  Updated environment lockfiles (SciTools#5211)
  update ci locks location (SciTools#5228)
  Fixes to _discontiguity_in_bounds (attempt 2) (SciTools#4975)
  Finalises Lazy Data documentation (SciTools#5137)
  Modernize and simplify iris.analysis._Groupby (SciTools#5015)
  clarity on whatsnew entry contributors (SciTools#5240)
  Handle derived coordinates correctly in `concatenate` (SciTools#5096)
  Use real array for data of of small netCDF variables. (SciTools#5229)
  Bump scitools/workflows from 2023.04.1 to 2023.04.2 (SciTools#5236)
  fixing whatsnew entry
  remove results creation commit from blame
  configure codecov
  adding a whatsnew entry
  Replacing numpy legacy printing with array2string and remaking results for dependent tests
  Adding a whatsnew entry for 5224 (SciTools#5234)
  Cf cell method (SciTools#5224)
  Bump scitools/workflows from 2023.03.3 to 2023.04.1 (SciTools#5231)
  [pre-commit.ci] pre-commit autoupdate (SciTools#5230)
  Bump scitools/workflows from 2023.03.2 to 2023.03.3 (SciTools#5227)
  raise dask min pin (SciTools#5225)
  ...
@codecov
Copy link

codecov bot commented Apr 13, 2023

Codecov Report

Patch and project coverage have no change.

Comparison is base (b0808e0) 89.31% compared to head (927e545) 89.31%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #5127   +/-   ##
=======================================
  Coverage   89.31%   89.31%           
=======================================
  Files          88       88           
  Lines       22279    22279           
  Branches     4881     4881           
=======================================
  Hits        19898    19898           
  Misses       1635     1635           
  Partials      746      746           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@tkknight tkknight requested a review from bjlittle April 17, 2023 10:50
@tkknight tkknight marked this pull request as ready for review April 17, 2023 10:50
@tkknight tkknight marked this pull request as draft April 17, 2023 10:57
@tkknight tkknight marked this pull request as ready for review April 17, 2023 16:20
Copy link
Member

@bjlittle bjlittle left a comment

Choose a reason for hiding this comment

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

@tkknight Another awesome step forward for the docs.

Superb! 🚀 🥳 🍻

@bjlittle bjlittle merged commit 7b6fa46 into SciTools:main Apr 18, 2023
tkknight added a commit to tkknight/iris that referenced this pull request Apr 18, 2023
* upstream/main:
  review actions
  update .git-blame-ignore-revs
  adopt codespell
  Adopt sphinx design (SciTools#5127)
  Bump scitools/workflows from 2023.04.2 to 2023.04.3 (SciTools#5253)
tkknight added a commit to tkknight/iris that referenced this pull request Apr 18, 2023
* upstream/main: (29 commits)
  review actions
  update .git-blame-ignore-revs
  adopt codespell
  Adopt sphinx design (SciTools#5127)
  Bump scitools/workflows from 2023.04.2 to 2023.04.3 (SciTools#5253)
  refresh manual pypi publish instructions (SciTools#5252)
  Updated environment lockfiles (SciTools#5250)
  removed bugfix section
  Make bm_runner location agnostic and include debugging. (SciTools#5247)
  Restore latest Whats New files.
  SciTools#5220 typo github.repository_owner. (SciTools#5248)
  Whats new updates for v3.5.0rc0. (SciTools#5246)
  libnetcdf <4.9 pin (SciTools#5242)
  update cf standard units (SciTools#5244)
  Updated environment lockfiles (SciTools#5211)
  update ci locks location (SciTools#5228)
  Fixes to _discontiguity_in_bounds (attempt 2) (SciTools#4975)
  Finalises Lazy Data documentation (SciTools#5137)
  Modernize and simplify iris.analysis._Groupby (SciTools#5015)
  clarity on whatsnew entry contributors (SciTools#5240)
  ...
tkknight added a commit to tkknight/iris that referenced this pull request Apr 22, 2023
* upstream/main: (61 commits)
  Updated environment lockfiles (SciTools#5270)
  Drop python3.8 support (SciTools#5269)
  build wheel from sdist, not src (SciTools#5266)
  Lazy netcdf saves (SciTools#5191)
  move setup.cfg to pyproject.toml (SciTools#5262)
  Support Python 3.11 (SciTools#5226)
  Remove Resolve test workaround (SciTools#5267)
  add missing whatsnew entry (SciTools#5265)
  make help (SciTools#5258)
  automate pypi manifest checking (SciTools#5259)
  drop sphinxcontrib-napoleon (SciTools#5263)
  add missing test result data (SciTools#5260)
  fix indentation and remove ref to ssstack (SciTools#5256)
  review actions
  update .git-blame-ignore-revs
  adopt codespell
  Adopt sphinx design (SciTools#5127)
  Bump scitools/workflows from 2023.04.2 to 2023.04.3 (SciTools#5253)
  refresh manual pypi publish instructions (SciTools#5252)
  Updated environment lockfiles (SciTools#5250)
  ...
@bjlittle bjlittle mentioned this pull request May 9, 2023
@tkknight tkknight deleted the adopt_sphinx_design branch November 9, 2023 15:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

sphinx-panels package no longer maintained
2 participants