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

Fixed sia2-remote tests #338

Merged
merged 1 commit into from
Jun 3, 2022
Merged

Fixed sia2-remote tests #338

merged 1 commit into from
Jun 3, 2022

Conversation

andamian
Copy link
Contributor

@andamian andamian commented Jun 3, 2022

Fix for #333

All these tests limit the number of rows returned through maxrec and the warning is legit. Changed to ignore it in these tests

@codecov
Copy link

codecov bot commented Jun 3, 2022

Codecov Report

Merging #338 (03fcffa) into main (a8aa366) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #338   +/-   ##
=======================================
  Coverage   78.28%   78.28%           
=======================================
  Files          46       46           
  Lines        5493     5493           
=======================================
  Hits         4300     4300           
  Misses       1193     1193           

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

@andamian andamian merged commit 197cd38 into astropy:main Jun 3, 2022
@andamian andamian deleted the fix_sia2 branch June 3, 2022 18:49
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.

None yet

2 participants