Skip to content

fix duplicate definitions in tornado #2424

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 1 commit into from
Sep 3, 2018
Merged

Conversation

JelleZijlstra
Copy link
Member

Fixes #2423.

I followed https://github.com/tornadoweb/tornado/blob/master/tornado/httpclient.py and kept the @property where it exists in the implementation and the attribute where it doesn't.

Fixes #2423.

I followed https://github.com/tornadoweb/tornado/blob/master/tornado/httpclient.py and kept the `@property` where it exists in the implementation and the attribute where it doesn't.
Copy link
Member

@ilevkivskyi ilevkivskyi left a comment

Choose a reason for hiding this comment

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

Thanks for fixing this!

@ilevkivskyi
Copy link
Member

What is the policy for situations when travis-ci/pr is green but travis-ci/push is red? (because the latter doesn't merge master before tests) I think this could be merged because the mypy fix has landed in master.

@JelleZijlstra JelleZijlstra merged commit 2e1c816 into master Sep 3, 2018
@JelleZijlstra
Copy link
Member Author

Yes let's just merge it if we're confident it won't break master.

@JelleZijlstra JelleZijlstra deleted the JelleZijlstra-patch-1 branch September 3, 2018 23:40
cocoatomo pushed a commit to cocoatomo/typeshed that referenced this pull request Sep 8, 2018
Fixes python#2423.

I followed https://github.com/tornadoweb/tornado/blob/master/tornado/httpclient.py and kept the `@property` where it exists in the implementation and the attribute where it doesn't.
yedpodtrzitko pushed a commit to yedpodtrzitko/typeshed that referenced this pull request Jan 23, 2019
Fixes python#2423.

I followed https://github.com/tornadoweb/tornado/blob/master/tornado/httpclient.py and kept the `@property` where it exists in the implementation and the attribute where it doesn't.
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.

2 participants