Skip to content

Clean sphinx makefile#573

Merged
dstansby merged 2 commits intozarr-developers:mainfrom
dstansby:clean-sphinx-makefile
Sep 18, 2024
Merged

Clean sphinx makefile#573
dstansby merged 2 commits intozarr-developers:mainfrom
dstansby:clean-sphinx-makefile

Conversation

@dstansby
Copy link
Copy Markdown
Contributor

@dstansby dstansby commented Sep 2, 2024

This cleans up the spinx Makefile to the latest version shipped with Sphinx. This reduces the size of it quite a bit, and fixes #457

@dstansby dstansby requested a review from a team September 18, 2024 08:18
@codecov
Copy link
Copy Markdown

codecov bot commented Sep 18, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 99.91%. Comparing base (c60f80c) to head (fd6d204).
Report is 68 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #573   +/-   ##
=======================================
  Coverage   99.91%   99.91%           
=======================================
  Files          59       59           
  Lines        2334     2334           
=======================================
  Hits         2332     2332           
  Misses          2        2           

@dstansby dstansby enabled auto-merge (squash) September 18, 2024 19:23
@dstansby dstansby merged commit 652a58e into zarr-developers:main Sep 18, 2024
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.

Local doc build claims sphinx not installed, when it is

2 participants