docs: Add sphinx api docs for aggregation, dbt, and openlineage packages#6271
Merged
ntkathole merged 2 commits intofeast-dev:masterfrom Apr 14, 2026
Merged
docs: Add sphinx api docs for aggregation, dbt, and openlineage packages#6271ntkathole merged 2 commits intofeast-dev:masterfrom
ntkathole merged 2 commits intofeast-dev:masterfrom
Conversation
![devin-ai-integration[bot]](https://avatars.githubusercontent.com/in/811515?s=60&v=4){kind=small}
This comment was marked as resolved.
This comment was marked as resolved.
Sorry, something went wrong.
korbonits
changed the title
korbonits
changed the title
This comment was marked as resolved.
This comment was marked as resolved.
Sorry, something went wrong.
korbonits
requested review from
ejscribner,
robhowley and
tokoko
and removed request for
a team
Add missing RST files for feast.aggregation (including tiling sub-package), feast.dbt, and feast.openlineage. Also add feast.diff.apply_progress which was missing from feast.diff.rst. Update feast.rst toctree to include all three new top-level packages. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com> Signed-off-by: Alex Korbonits <alex@korbonits.com>
No new warning categories are introduced by adding feast.aggregation, feast.dbt, and feast.openlineage to the Sphinx docs. The 10 new "more than one target found" warnings follow the same pre-existing pattern (8 already present) caused by feast.__init__ re-exporting classes under multiple paths. Fixes applied: - Remove :undoc-members: from dataclass automodules (Aggregation, IRMetadata, OpenLineageConfig) to prevent duplicate attribute docs - Document re-exporting __init__ packages at package level only to avoid duplicate descriptions from submodule entries - Fix markdown-style code fences in openlineage/__init__.py docstring - Fix RST formatting in client.py Example section (Example::) and emitter.py ASCII art diagram (literal block) - Fix repo_config.py field docstring that triggered an ambiguous OpenLineageConfig cross-reference Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com> Signed-off-by: Alex Korbonits <alex@korbonits.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
feast.aggregation.rstandfeast.aggregation.tiling.rstfor the aggregation package and its tiling sub-packagefeast.dbt.rstfor the dbt integration (codegen, mapper, parser)feast.openlineage.rstfor the OpenLineage integration (client, config, emitter, facets, mappers)feast.diff.apply_progressmodule tofeast.diff.rstfeast.rsttoctree to include all three new top-level packagesSphinx docs were falling behind as new modules were added. This addresses the gaps noted in #6266.
Test plan
make build-sphinxlocally and confirm the new packages appear in the generated HTML docs🤖 Generated with Claude Code