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

Exclude pynwb retinotopy from sphinx apidoc #1846

Merged
merged 6 commits into from
Feb 16, 2024

Conversation

rly
Copy link
Contributor

@rly rly commented Feb 10, 2024

Deprecatingimport pynwb.retinotopy results in an ImportError during sphinx docs generation. This PR tries to exclude that module from sphinx processing

Copy link

codecov bot commented Feb 10, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (d59fe07) 91.80% compared to head (e76965d) 91.80%.

Additional details and impacted files
@@                Coverage Diff                @@
##           nwb-schema-2.7.0    #1846   +/-   ##
=================================================
  Coverage             91.80%   91.80%           
=================================================
  Files                    27       27           
  Lines                  2625     2625           
  Branches                701      701           
=================================================
  Hits                   2410     2410           
  Misses                  142      142           
  Partials                 73       73           
Flag Coverage Δ
integration 70.89% <ø> (ø)
unit 83.88% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@rly rly marked this pull request as ready for review February 16, 2024 01:02
@rly rly requested a review from oruebel February 16, 2024 01:02
@rly
Copy link
Contributor Author

rly commented Feb 16, 2024

The sphinx ImportError issue has now been resolved. The check-sphinx-links workflow is now failing because of an unrelated SSL certificate issue with incf.org.

@rly rly changed the title [WIP] Exclude pynwb retinotopy from sphinx Exclude pynwb retinotopy from sphinx apidoc Feb 16, 2024
@rly rly merged commit 90d6b1d into nwb-schema-2.7.0 Feb 16, 2024
22 of 23 checks passed
@rly rly deleted the sphinx-retinotopy-fix branch February 16, 2024 01:05
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