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

Fix bug in ibe list_missions() #1426

Closed
wants to merge 5 commits into from

Conversation

odysseus9672
Copy link
Contributor

@odysseus9672 odysseus9672 commented Apr 25, 2019

Resubmitting bug fix I accidentally deleted. I apologize.

This fixes issue #1423

Resubmitting bug fix I accidentally deleted.
@odysseus9672 odysseus9672 changed the title Fix bug in list_missions() Fix bug in ibe list_missions() Apr 25, 2019
@codecov
Copy link

codecov bot commented Apr 25, 2019

Codecov Report

Merging #1426 into master will not change coverage.
The diff coverage is 100%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1426   +/-   ##
=======================================
  Coverage   64.89%   64.89%           
=======================================
  Files         165      165           
  Lines       12860    12860           
=======================================
  Hits         8346     8346           
  Misses       4514     4514
Impacted Files Coverage Δ
astroquery/ibe/core.py 78.57% <100%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 06d6e67...0766865. Read the comment docs.

@pep8speaks
Copy link

pep8speaks commented Apr 25, 2019

Hello @odysseus9672! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

There are currently no PEP 8 issues detected in this Pull Request. Cheers! 🍻

Comment last updated at 2019-04-25 02:47:05 UTC

@odysseus9672
Copy link
Contributor Author

Superceded by #1430

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