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

TST: fixes for 2023 aug #2800

Merged
merged 3 commits into from
Aug 9, 2023
Merged

TST: fixes for 2023 aug #2800

merged 3 commits into from
Aug 9, 2023

Conversation

bsipocz
Copy link
Member

@bsipocz bsipocz commented Aug 7, 2023

These are fixes for current test failures. Most of them are just updates, but

@bsipocz bsipocz added the testing label Aug 7, 2023
@bsipocz bsipocz added this to the v0.4.7 milestone Aug 7, 2023
@bsipocz bsipocz force-pushed the tst_fixes_for_23Aug branch from 833228b to ed8913e Compare August 8, 2023 18:36
@codecov
Copy link

codecov bot commented Aug 8, 2023

Codecov Report

Merging #2800 (4b109a7) into main (3f16e62) will not change coverage.
The diff coverage is 50.00%.

@@           Coverage Diff           @@
##             main    #2800   +/-   ##
=======================================
  Coverage   66.17%   66.17%           
=======================================
  Files         235      235           
  Lines       18109    18109           
=======================================
  Hits        11983    11983           
  Misses       6126     6126           
Files Changed Coverage Δ
astroquery/lamda/core.py 71.57% <0.00%> (ø)
astroquery/sdss/core.py 92.97% <100.00%> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@bsipocz bsipocz force-pushed the tst_fixes_for_23Aug branch from ed8913e to dcafc78 Compare August 8, 2023 20:28
@bsipocz bsipocz force-pushed the tst_fixes_for_23Aug branch from dcafc78 to 4b109a7 Compare August 9, 2023 17:23
@bsipocz bsipocz merged commit b448335 into astropy:main Aug 9, 2023
@bsipocz bsipocz deleted the tst_fixes_for_23Aug branch December 8, 2023 18:17
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.

1 participant