Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Fix isort issues #309

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
Nov 27, 2019
Merged

Fix isort issues #309

merged 7 commits into from
Nov 27, 2019

Conversation

ocelotl
Copy link
Contributor

@ocelotl ocelotl commented Nov 27, 2019

Fixes #308

@ocelotl
Copy link
Contributor Author

ocelotl commented Nov 27, 2019

All these "fixes" were generated by just running isort recursively in the affected directories. Nevertheless, some of them seem odd regarding how the imports are being arranged. Mostly, I am curious to see what happens with the CI of this PR, please don't merge yet.

@codecov-io
Copy link

codecov-io commented Nov 27, 2019

Codecov Report

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

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #309   +/-   ##
=======================================
  Coverage   85.76%   85.76%           
=======================================
  Files          33       33           
  Lines        1609     1609           
  Branches      180      180           
=======================================
  Hits         1380     1380           
  Misses        182      182           
  Partials       47       47

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 a42b063...3fc3394. Read the comment docs.

@c24t c24t merged commit 2bb6a17 into open-telemetry:master Nov 27, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

tox -e lint is failing
4 participants