Skip to content

Add in_place argument to FormatFile.#158

Merged
bwendling merged 1 commit intogoogle:masterfrom
hayd:inplace_FormatFile
Jun 17, 2015
Merged

Add in_place argument to FormatFile.#158
bwendling merged 1 commit intogoogle:masterfrom
hayd:inplace_FormatFile

Conversation

@hayd
Copy link
Copy Markdown
Contributor

@hayd hayd commented Jun 16, 2015

(For use as a module.)

(For use as a module.)

Corrects the docstring of FormatCode to say that it raises if there is an
IOError. Previously this claimed it returned (None, encoding) which was
not correct. Now (None, encoding) is used if the FormatFile was used with
in_place.
@hayd hayd force-pushed the inplace_FormatFile branch from 359852f to 921ce03 Compare June 16, 2015 23:58
@hayd
Copy link
Copy Markdown
Contributor Author

hayd commented Jun 17, 2015

@gwelymernans Fixed. (As you can see our previous discussion has been wiped from the PR!)

@bwendling
Copy link
Copy Markdown
Member

Oy! That is a nasty bug...

bwendling added a commit that referenced this pull request Jun 17, 2015
Add in_place argument to FormatFile.
@bwendling bwendling merged commit 88ddeb5 into google:master Jun 17, 2015
hayd added a commit to hayd/yapf that referenced this pull request Jun 17, 2015
Also fix a typo from updating the README in google#157.

Note: in_place argument introduced in google#158.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants