Skip to content

Remove deprecated NASA Exoplanet Archive code#2431

Merged
bsipocz merged 2 commits into
astropy:mainfrom
eerovaher:rm-nasa-exoplanet-deprecated
Jun 2, 2022
Merged

Remove deprecated NASA Exoplanet Archive code#2431
bsipocz merged 2 commits into
astropy:mainfrom
eerovaher:rm-nasa-exoplanet-deprecated

Conversation

@eerovaher

Copy link
Copy Markdown
Member

The removed code was deprecated by #1700 in v0.4.1 almost two years ago.

eerovaher added 2 commits June 2, 2022 22:46
The removed code was deprecated in v0.4.1 almost two years ago.
@codecov

codecov Bot commented Jun 2, 2022

Copy link
Copy Markdown

Codecov Report

Merging #2431 (f53f956) into main (eceb776) will decrease coverage by 0.04%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #2431      +/-   ##
==========================================
- Coverage   62.97%   62.92%   -0.05%     
==========================================
  Files         133      133              
  Lines       17290    17271      -19     
==========================================
- Hits        10888    10868      -20     
- Misses       6402     6403       +1     
Impacted Files Coverage Δ
...roquery/ipac/nexsci/nasa_exoplanet_archive/core.py 57.26% <ø> (-3.51%) ⬇️

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

@bsipocz bsipocz left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Super nitpick (that I'll fix in release): you need to add the PR number in the changelog.

Very handy small script to get it before opening a PR (so to avoid rerunning the CI for the changelog): https://github.com/astropy/astropy-tools/blob/main/next_pr_number.py

@bsipocz bsipocz merged commit d254369 into astropy:main Jun 2, 2022
@bsipocz

bsipocz commented Jun 2, 2022

Copy link
Copy Markdown
Member

Thanks @eerovaher!

@eerovaher eerovaher deleted the rm-nasa-exoplanet-deprecated branch June 2, 2022 22:36
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.

3 participants