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

[BUG] [Docs] Allow source code discovery to fail silently for pyo3-defined classes when generating docs. #1430

Merged
merged 1 commit into from
Sep 25, 2023

Conversation

clarkzinzow
Copy link
Contributor

This PR allows source code discovery to silently fail for pyo3-defined classes when generating docs, since I believe that the source code for pyo3-defined classes isn't easily available anyway.

@clarkzinzow clarkzinzow requested a review from jaychia September 25, 2023 18:04
@github-actions github-actions bot added bug Something isn't working documentation Improvements or additions to documentation labels Sep 25, 2023
@codecov
Copy link

codecov bot commented Sep 25, 2023

Codecov Report

Merging #1430 (82cbce6) into main (84c727d) will decrease coverage by 13.25%.
Report is 11 commits behind head on main.
The diff coverage is 0.00%.

Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff             @@
##             main    #1430       +/-   ##
===========================================
- Coverage   87.78%   74.53%   -13.25%     
===========================================
  Files          60       60               
  Lines        6032     6035        +3     
===========================================
- Hits         5295     4498      -797     
- Misses        737     1537      +800     
Files Coverage Δ
daft/__init__.py 26.47% <0.00%> (ø)

... and 16 files with indirect coverage changes

@clarkzinzow clarkzinzow merged commit ecb596a into main Sep 25, 2023
22 of 24 checks passed
@clarkzinzow clarkzinzow deleted the clark/fix-docs-source-discovery-pyo3 branch September 25, 2023 18:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants