Skip to content

Autodoc - #39

Open
jeanjacquemier wants to merge 1 commit into
mainfrom
autodoc
Open

jeanjacquemier wants to merge 1 commit into
mainfrom
autodoc

Conversation

@jeanjacquemier

@jeanjacquemier jeanjacquemier commented Oct 6, 2026 •

Copy link
Copy Markdown
Collaborator

Add the documentation with Sphinx and the corresponding mechanisms
- Add a logo to the README
- Add clp_covariance and clp_firecrown confirguration options
- Add an overview
- Add an installation doc:
- Add doc generation with CI
- Use github Pages withgithub actions to publish the docs
- add deployment on autodoc branch before merging
- Add some github badges
- Add LSST DESC software dependencie to doc

Comment thread .github/workflows/docs.yml

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot review overview

🟡 Changes recommended

The documentation contains invalid configuration guidance, lacks functional autodoc, and includes build and deployment defects.

Review effort: Balanced
Findings: 1 High severity · 6 Medium severity · 6 Low severity

Open (13)
What changed in this PR

Introduces a Sphinx documentation site, configuration guides, and automated validation/deployment.

Changes:

  • Adds overview, installation, covariance, and Firecrown documentation.
  • Adds Sphinx configuration and dependencies.
  • Adds documentation CI, GitHub Pages deployment, and README branding.
File Description
README.md Adds the CLPipe logo.
docs/​source/​overview.rst Adds the project overview.
docs/​source/​installation.rst Adds installation and testing instructions.
docs/​source/​clp_firecrown.rst Documents Firecrown configuration.
docs/​source/​clp_covariance.rst Documents covariance configuration.
docs/​requirements.txt Declares documentation dependencies.
docs/​Makefile Adds Sphinx build targets.
docs/​index.rst Defines documentation navigation.
docs/​conf.py Configures Sphinx and its theme.
.gitignore Ignores generated documentation.
.github/​workflows/​docs.yml Builds and deploys documentation.
.github/​workflows/​ci.yml Adds documentation validation.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread docs/Makefile Outdated
Comment thread .github/workflows/docs.yml
Comment thread docs/Makefile Outdated
Comment thread docs/conf.py Outdated
Comment thread docs/source/clp_covariance.rst Outdated
Comment thread docs/index.rst Outdated
Comment thread docs/source/clp_covariance.rst Outdated
Comment on lines +161 to +162
This is handled by:
extract_data_covariance()

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Comment thread docs/source/installation.rst Outdated
Comment thread docs/source/installation.rst Outdated
Comment thread docs/source/overview.rst Outdated
@coveralls

coveralls commented Oct 7, 2026 •

Copy link
Copy Markdown

Coverage Report for CI Build 37614240398

Warning

No base build found for commit 3a6cd70 on main.
Coverage changes can't be calculated without a base build.
If a base build is processing, this comment will update automatically when it completes.

Coverage: 42.255%

Details

  • Patch coverage: No coverable lines changed in this PR.

Uncovered Changes

No uncovered changes found.

Coverage Regressions

Requires a base build to compare against. How to fix this →


Coverage Stats

Coverage Status
Relevant Lines: 1020
Covered Lines: 431
Line Coverage: 42.25%
Coverage Strength: 0.85 hits per line

💛 - Coveralls

@codecov-commenter

Copy link
Copy Markdown

Welcome to Codecov 🎉

Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.

Thanks for integrating Codecov - We've got you covered ☂️

Comment thread .github/workflows/docs.yml
    - Add a logo to the README
    - Add clp_covariance and clp_firecrown confirguration options
    - Add an overview
    - Add an installation doc:
    - Add doc generation with CI
    - Use github Pages withgithub actions to publish the docs
    - add deployment on autodoc branch before merging
    - Add some github badges
    - Add LSST DESC software dependencie to doc
    - Add sphinx copy button extension
    - Add coveralls command
    - Add API doc

This branch was successfully deployed

1 active deployment
github-pages — 5424be0d Deployed Oct 12, 2026 by jeanjacquemier via deploy #50
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.

5 participants