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

[wip] Set up yapf to do auto-formatting #221

Merged
merged 3 commits into from
Aug 4, 2017
Merged

Conversation

njsmith
Copy link
Member

@njsmith njsmith commented Jun 15, 2017

This is the preparatory stuff -- at the right moment we can rebase
this and then run 'yapf -rip setup.py trio' to reformat everything.

@njsmith njsmith force-pushed the yapf branch 2 times, most recently from 4f94a8e to 3e68826 Compare June 15, 2017 09:21
@codecov
Copy link

codecov bot commented Jun 15, 2017

Codecov Report

Merging #221 into master will decrease coverage by 0.01%.
The diff coverage is 97.87%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #221      +/-   ##
==========================================
- Coverage   99.15%   99.13%   -0.02%     
==========================================
  Files          67       67              
  Lines        9190     9213      +23     
  Branches      656      663       +7     
==========================================
+ Hits         9112     9133      +21     
  Misses         62       62              
- Partials       16       18       +2
Impacted Files Coverage Δ
trio/tests/test_timeouts.py 100% <ø> (ø) ⬆️
trio/_core/tests/test_unbounded_queue.py 100% <ø> (ø) ⬆️
trio/_core/__init__.py 100% <ø> (ø) ⬆️
trio/_core/_unbounded_queue.py 100% <ø> (ø) ⬆️
trio/tests/test_abc.py 100% <ø> (ø) ⬆️
trio/_path.py 100% <ø> (ø) ⬆️
trio/_open_tcp_stream.py 96.66% <ø> (ø) ⬆️
trio/_ssl_stream_helpers.py 100% <ø> (ø) ⬆️
...tests/test_multierror_scripts/custom_excepthook.py 100% <ø> (ø) ⬆️
trio/_core/_windows_cffi.py 91.3% <ø> (ø) ⬆️
... and 53 more

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 fef2215...adf6632. Read the comment docs.

@njsmith njsmith mentioned this pull request Jun 17, 2017
Small tweaks to make yapf work better. No semantic changes.
This is a whitespace-only commit. I ran 'yapf -rip setup.py trio'.
@njsmith njsmith merged commit 027d374 into python-trio:master Aug 4, 2017
@njsmith njsmith deleted the yapf branch August 4, 2017 04:15
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.

1 participant