Skip to content

Conversation

@glatterf42
Copy link
Member

@glatterf42 glatterf42 commented Nov 6, 2023

Closes #758. This PR addresses some of the docs-build process errors described in that issue. See there for more details.

How to review

  • Read the diff and note that the CI checks all pass.
  • Check the RTD-built documentation and check the warnings/that they don't bother us (generally on the api, reporting, and macro pages)

PR checklist

  • Continuous integration checks all ✅
  • [ ] Add or expand tests; coverage checks both ✅ No functionality changes.
  • Add, expand, or update documentation.
  • [ ] Update release notes. No user-facing changes.

@glatterf42 glatterf42 added bug Doesn't work as advertised/unintended effects docs Documentation labels Nov 6, 2023
@codecov
Copy link

codecov bot commented Nov 6, 2023

Codecov Report

Merging #759 (8cc2c58) into main (f360dc7) will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff          @@
##            main    #759   +/-   ##
=====================================
  Coverage   94.3%   94.3%           
=====================================
  Files         43      43           
  Lines       3474    3474           
=====================================
  Hits        3279    3279           
  Misses       195     195           

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Doesn't work as advertised/unintended effects docs Documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Resolving documentation build warnings

1 participant