Skip to content

Fix mypy tests for contextlib change in typeshed PR 3083#6958

Merged
ilevkivskyi merged 1 commit intopython:masterfrom
RafiB:add-decorator-tests
Jul 2, 2019
Merged

Fix mypy tests for contextlib change in typeshed PR 3083#6958
ilevkivskyi merged 1 commit intopython:masterfrom
RafiB:add-decorator-tests

Conversation

@RafiB
Copy link
Contributor

@RafiB RafiB commented Jun 8, 2019

python/typeshed#3083 fixed an issue in the contextlib stub file, which broke an existing mypy test that was written based off the incorrect stub file.

NOTE:
This commit will have to synced with that change to fix the tests + add new decorator tests.

@RafiB RafiB force-pushed the add-decorator-tests branch 4 times, most recently from 427f28c to d360f91 Compare June 22, 2019 05:29
@RafiB RafiB changed the title Add tests for decorator stub file introduced in typeshed PR 3038 Fix mypy tests for contextlib change in typeshed PR 3038 Jun 22, 2019
@RafiB RafiB changed the title Fix mypy tests for contextlib change in typeshed PR 3038 Fix mypy tests for contextlib change in typeshed PR 3083 Jun 22, 2019
@RafiB RafiB force-pushed the add-decorator-tests branch from d360f91 to a19dcdb Compare June 22, 2019 05:49
@srittau
Copy link
Contributor

srittau commented Jul 1, 2019

@JelleZijlstra @ilevkivskyi I'd like to get the corresponding typeshed fix (python/typeshed#3083) merged. What's the status of this PR?

@JelleZijlstra
Copy link
Member

Merging this will require a complicated dance of merging the typeshed PR + syncing the typeshed submodule in mypy + merging this PR. I can do it at some point but don't have time right now.

@ilevkivskyi
Copy link
Member

I can do the typeshed dance now.

@ilevkivskyi ilevkivskyi mentioned this pull request Jul 2, 2019
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.

I prepared the typeshed sync. I will merge this right after it lands.

@ilevkivskyi
Copy link
Member

I have restarted all builds, should be green now.

@ilevkivskyi ilevkivskyi merged commit d1b6f7f into python:master Jul 2, 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.

4 participants