Skip to content

MNT: cleanup deprecations and obsolte checks#2389

Merged
bsipocz merged 6 commits into
astropy:mainfrom
bsipocz:pyvo_usage_cleanup
May 4, 2022
Merged

MNT: cleanup deprecations and obsolte checks#2389
bsipocz merged 6 commits into
astropy:mainfrom
bsipocz:pyvo_usage_cleanup

Conversation

@bsipocz

@bsipocz bsipocz commented May 3, 2022

Copy link
Copy Markdown
Member
  • PyVO is a required dependency now.

  • individual tests can be run with either python setup.py -t testfilename::testfunctionname or as pytest testfilename -k testfunctionname

  • 0.4.0 has been a while ago, deprecations are likely can be removed now

@bsipocz bsipocz added the cadc label May 3, 2022
@bsipocz bsipocz added this to the v0.4.7 milestone May 3, 2022
@bsipocz bsipocz requested a review from andamian May 3, 2022 04:23
@codecov

codecov Bot commented May 3, 2022

Copy link
Copy Markdown

Codecov Report

Merging #2389 (3aa91b4) into main (105d232) will decrease coverage by 0.43%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main    #2389      +/-   ##
==========================================
- Coverage   63.31%   62.87%   -0.44%     
==========================================
  Files         133      133              
  Lines       17271    17246      -25     
==========================================
- Hits        10935    10844      -91     
- Misses       6336     6402      +66     
Impacted Files Coverage Δ
astroquery/cadc/core.py 80.37% <100.00%> (+3.49%) ⬆️
astroquery/nrao/core.py 24.57% <0.00%> (-46.46%) ⬇️

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

@andamian andamian left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Run the remote tests locally with more privileges as well as doctests and all passed.

@andamian

andamian commented May 3, 2022

Copy link
Copy Markdown

@bsipocz - thank you for doing this.

@bsipocz bsipocz merged commit 96f63eb into astropy:main May 4, 2022
@bsipocz

bsipocz commented May 4, 2022

Copy link
Copy Markdown
Member Author

@bsipocz - thank you for doing this.

No worries, PRs with negative lines bring me real joy :)

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants