Skip to content

Upgrade pytest-asyncio to v0.10.0 #214

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

Merged
merged 7 commits into from
Jan 8, 2019
Merged

Upgrade pytest-asyncio to v0.10.0 #214

merged 7 commits into from
Jan 8, 2019

Conversation

maxalbert
Copy link
Contributor

Closes #213

I have:

  • Formatted any Python files with black
  • Brought the branch up to date with master
  • Added any relevant Github labels
  • Added tests for any new additions
  • Added or updated any relevant documentation
  • Added an Architectural Decision Record (ADR), if appropriate
  • Added an MPLv2 License Header if appropriate

Description

I have re-locked the Pipfiles so pytest-asyncio is now set to v0.10.0. Only three Pipfile.lock files were affected but I have re-locked the others for good measure.

@maxalbert maxalbert added tests Issues relating to tests dependencies Pull requests that update a dependency file CI Issues related to continuous integration & circleci labels Jan 8, 2019
@greenape
Copy link
Member

greenape commented Jan 8, 2019

This closes #207, #209, #204 as well right?

Copy link
Member

@greenape greenape left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Grand. Thanks @maxalbert

@greenape greenape added the ready-to-merge Label indicating a PR is OK to automerge label Jan 8, 2019
@maxalbert
Copy link
Contributor Author

Yes, it should close #207, #209, #204 too.

@codecov
Copy link

codecov bot commented Jan 8, 2019

Codecov Report

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

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #214   +/-   ##
=======================================
  Coverage   89.68%   89.68%           
=======================================
  Files          73       73           
  Lines        4413     4413           
  Branches      519      519           
=======================================
  Hits         3958     3958           
  Misses        349      349           
  Partials      106      106

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 89fc27a...019da1c. Read the comment docs.

1 similar comment
@codecov
Copy link

codecov bot commented Jan 8, 2019

Codecov Report

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

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #214   +/-   ##
=======================================
  Coverage   89.68%   89.68%           
=======================================
  Files          73       73           
  Lines        4413     4413           
  Branches      519      519           
=======================================
  Hits         3958     3958           
  Misses        349      349           
  Partials      106      106

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 89fc27a...019da1c. Read the comment docs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI Issues related to continuous integration & circleci dependencies Pull requests that update a dependency file ready-to-merge Label indicating a PR is OK to automerge tests Issues relating to tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants