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

DOC - Fix most documentation build warnings #539

Merged
merged 3 commits into from
Nov 20, 2024

Conversation

Valentin-Laurent
Copy link
Collaborator

@Valentin-Laurent Valentin-Laurent commented Nov 20, 2024

Description

Fixing most warnings appearing when building the documentation (ie running make doc). The goal was to start from a clean state when working on the v1 documentation. This PR may break a few things, but mostly it will guarantee an overall proper display of documentation and especially docstrings, IDE included.

I didn't manage to fix all the warnings related to the file mapie/conformity_scores/regression.py. It seems pretty broken: when I delete problematic docstrings, new warning appears, making me think something is kind of corrupting the docstring parsing at some point.

Fixes #(issue)

Type of change

Please remove options that are irrelevant.

  • Bug fix (non-breaking change which fixes an issue)
  • This change requires a documentation update

Checklist

  • I have read the contributing guidelines
  • I have updated the HISTORY.rst and AUTHORS.rst files
  • Linting passes successfully : make lint
  • Typing passes successfully : make type-check
  • Unit tests pass successfully : make tests
  • Coverage is 100% : make coverage
  • Documentation builds successfully : make doc

@Valentin-Laurent Valentin-Laurent merged commit 58e839c into master Nov 20, 2024
8 checks passed
@Valentin-Laurent Valentin-Laurent deleted the doc-fix-warnings branch November 20, 2024 15:43
@Valentin-Laurent Valentin-Laurent mentioned this pull request Nov 21, 2024
7 tasks
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.

2 participants