Skip to content

Fix flake8 io deadlock#757

Merged
ccordoba12 merged 3 commits into
palantir:developfrom
rdugan:flake8-io-deadlock-fix
May 10, 2020
Merged

Fix flake8 io deadlock#757
ccordoba12 merged 3 commits into
palantir:developfrom
rdugan:flake8-io-deadlock-fix

Conversation

@rdugan

@rdugan rdugan commented Feb 29, 2020

Copy link
Copy Markdown
Contributor

Second attempt at fix to address #751, including test updates.

Fixes #751.

@ccordoba12 ccordoba12 changed the title Flake8 io deadlock fix Fix flake8 io deadlock May 10, 2020

@ccordoba12 ccordoba12 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks for your help @rdugan! And sorry for the delay in merging your work.

@ccordoba12 ccordoba12 merged commit 22251fa into palantir:develop May 10, 2020
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.

flake8 plugin using fragile stderr.read() call

2 participants