Skip to content

Convert about page to sphinx-design#93

Merged
RNKuhns merged 5 commits into
sktime:mainfrom
RNKuhns:sphinx_design
Dec 31, 2022
Merged

Convert about page to sphinx-design#93
RNKuhns merged 5 commits into
sktime:mainfrom
RNKuhns:sphinx_design

Conversation

@RNKuhns
Copy link
Copy Markdown
Contributor

@RNKuhns RNKuhns commented Dec 30, 2022

Reference Issues/PRs

Fixes #50

What does this implement/fix? Explain your changes.

Uses sphinx-design instead of sphinx-panel in the project's Sphinx docs. Sphinx-design provides a nicer design aesthetic and I believe sphinx-panel is no longer maintained.

Does your contribution introduce a new dependency? If yes, which one?

sphinx-design is a new documentatino dependency (instead of sphinx-panel).

What should a reviewer concentrate their feedback on?

Do the affected documentation pages render correctly and are the panels "clickable".

@RNKuhns RNKuhns requested a review from fkiraly December 30, 2022 08:09
@RNKuhns RNKuhns added this to the Release 0.3.0 milestone Dec 30, 2022
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Dec 30, 2022

Codecov Report

Merging #93 (d827f87) into main (d9eb9f9) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #93   +/-   ##
=======================================
  Coverage   60.86%   60.86%           
=======================================
  Files          23       23           
  Lines        1477     1477           
=======================================
  Hits          899      899           
  Misses        578      578           

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

Copy link
Copy Markdown
Contributor

@fkiraly fkiraly left a comment

Choose a reason for hiding this comment

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

tested it, links and panels work.

@RNKuhns RNKuhns merged commit 853d5ea into sktime:main Dec 31, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[DOC] Switch from sphinx-panels to sphinx-design extension

3 participants