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

Add some helpful docstrings to utils.py #1230

Merged
merged 2 commits into from
May 15, 2019
Merged

Conversation

vEpiphyte
Copy link
Contributor

Type hinting aware editors are happier with these :)

@vEpiphyte vEpiphyte added this to the v0.1.5 milestone May 15, 2019
@vEpiphyte vEpiphyte self-assigned this May 15, 2019
@codecov
Copy link

codecov bot commented May 15, 2019

Codecov Report

Merging #1230 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@          Coverage Diff           @@
##           master   #1230   +/-   ##
======================================
  Coverage      94%     94%           
======================================
  Files         126     126           
  Lines       16151   16151           
======================================
  Hits        15183   15183           
  Misses        968     968
Impacted Files Coverage Δ
synapse/tests/utils.py 92.26% <ø> (ø) ⬆️
synapse/cortex.py 90.62% <ø> (ø) ⬆️

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 2251aab...8b5e1bb. Read the comment docs.

@invisig0th invisig0th merged commit 9c70288 into master May 15, 2019
@vEpiphyte vEpiphyte deleted the test_util_docstrings branch May 15, 2019 23:15
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.

2 participants