Skip to content

stubtest: workaround mypyc's issue with vars - #8411

Merged
msullivan merged 1 commit into
python:masterfrom
hauntsaninja:stubtestmypyc
Feb 18, 2020
Merged

msullivan merged 1 commit into
python:masterfrom
hauntsaninja:stubtestmypyc

Conversation

@hauntsaninja

@hauntsaninja hauntsaninja commented Feb 16, 2020 •

Copy link
Copy Markdown
Collaborator

Follow up from #8380

Thanks for the fix! The cast works, but unfortunately we have one last issue: mypyc does not like the conditional definitions necessary to work around a bug in Python 3.5.2 and earlier (8fda109). Example stacktrace here: https://travis-ci.org/hauntsaninja/mypy/jobs/651008763
Long story short, this PR doesn't do anything, but maybe it will in the future!

@msullivan msullivan left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

We could get it working by sticking MaybeMissing[whatever] in quotes everywhere which wouldn't be too bad but it also isn't too important to compile this anyway.

Looks good.

@msullivan
msullivan merged commit 6efbff4 into python:master Feb 18, 2020
@hauntsaninja
hauntsaninja deleted the stubtestmypyc branch February 21, 2020 08:16
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