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

Remove possible debug print statement? #2559

Merged
merged 1 commit into from
Oct 20, 2022
Merged

Conversation

adrn
Copy link
Member

@adrn adrn commented Oct 18, 2022

I've noticed this printed phrase whenever doing queries with the Gaia class and wondered if it was left in from debugging at some point. Can this be removed?

In some cases it can lead to distracting output, e.g.,
image

@codecov
Copy link

codecov bot commented Oct 18, 2022

Codecov Report

Merging #2559 (756f5a7) into main (5af5780) will decrease coverage by 0.00%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #2559      +/-   ##
==========================================
- Coverage   63.97%   63.97%   -0.01%     
==========================================
  Files         132      132              
  Lines       16984    16983       -1     
==========================================
- Hits        10866    10865       -1     
  Misses       6118     6118              
Impacted Files Coverage Δ
astroquery/utils/tap/xmlparser/jobSaxParser.py 96.87% <ø> (-0.04%) ⬇️

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

@bsipocz bsipocz added this to the v0.4.7 milestone Oct 20, 2022
@bsipocz bsipocz merged commit 2486495 into astropy:main Oct 20, 2022
@bsipocz
Copy link
Member

bsipocz commented Oct 20, 2022

Thanks!

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