Skip to content

Cadc documentation clean up#1942

Merged
bsipocz merged 5 commits into
astropy:mainfrom
tinuademargaret:cadc-doc-cleanup
Apr 22, 2022
Merged

Cadc documentation clean up#1942
bsipocz merged 5 commits into
astropy:mainfrom
tinuademargaret:cadc-doc-cleanup

Conversation

@tinuademargaret

Copy link
Copy Markdown
Contributor

No description provided.

@astropy-bot astropy-bot Bot added the cadc label Jan 13, 2021
@codecov

codecov Bot commented Jan 13, 2021

Copy link
Copy Markdown

Codecov Report

Merging #1942 (dd149f7) into main (115b0aa) will increase coverage by 0.11%.
The diff coverage is 56.20%.

@@            Coverage Diff             @@
##             main    #1942      +/-   ##
==========================================
+ Coverage   63.17%   63.29%   +0.11%     
==========================================
  Files         132      132              
  Lines       17189    17241      +52     
==========================================
+ Hits        10860    10913      +53     
+ Misses       6329     6328       -1     
Impacted Files Coverage Δ
astroquery/esa/iso/core.py 64.58% <ø> (ø)
astroquery/exceptions.py 0.00% <0.00%> (ø)
astroquery/gaia/core.py 71.94% <ø> (ø)
astroquery/image_cutouts/first/core.py 92.68% <ø> (ø)
astroquery/magpis/core.py 93.61% <ø> (ø)
astroquery/mast/services.py 79.56% <ø> (ø)
astroquery/atomic/core.py 35.77% <30.76%> (ø)
astroquery/utils/tap/taputils.py 67.85% <48.78%> (ø)
astroquery/alma/core.py 42.80% <55.55%> (ø)
astroquery/mast/missions.py 76.54% <75.00%> (ø)
... and 19 more

📣 Codecov can now indicate which changes are the most critical in Pull Requests. Learn more

@bsipocz bsipocz marked this pull request as draft February 25, 2022 06:45
@bsipocz bsipocz marked this pull request as ready for review April 22, 2022 18:54
@bsipocz bsipocz added this to the v0.4.7 milestone Apr 22, 2022
@bsipocz bsipocz merged commit 540c690 into astropy:main Apr 22, 2022
@bsipocz

bsipocz commented Apr 22, 2022

Copy link
Copy Markdown
Member

I've opted for skipping the examples that cause troubles, some of which look to be genuine bugs, see issues opened recently for the module. Otherwise good to have one more module's narrative docs tested.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants