Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Correctly report the port running aioconsole #124

Merged
merged 1 commit into from
Jul 9, 2018
Merged

Correctly report the port running aioconsole #124

merged 1 commit into from
Jul 9, 2018

Conversation

bmerry
Copy link
Contributor

@bmerry bmerry commented Jul 9, 2018

It was previously reporting the aiomonitor port.

It was previously reporting the aiomonitor port.
@codecov
Copy link

codecov bot commented Jul 9, 2018

Codecov Report

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

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #124   +/-   ##
=======================================
  Coverage   92.82%   92.82%           
=======================================
  Files           4        4           
  Lines         251      251           
  Branches       32       32           
=======================================
  Hits          233      233           
  Misses         11       11           
  Partials        7        7
Impacted Files Coverage Δ
aiomonitor/monitor.py 91.61% <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 3318369...1fe87bb. Read the comment docs.

@jettify
Copy link
Member

jettify commented Jul 9, 2018

good catch!

@jettify jettify merged commit 4d426fb into aio-libs:master Jul 9, 2018
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants