Navigation Menu

Skip to content
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

Run mypy on the manifest code. #16346

Merged
merged 2 commits into from Apr 24, 2019
Merged

Run mypy on the manifest code. #16346

merged 2 commits into from Apr 24, 2019

Conversation

Ms2ger
Copy link
Contributor

@Ms2ger Ms2ger commented Apr 15, 2019

No description provided.

@Ms2ger
Copy link
Contributor Author

Ms2ger commented Apr 15, 2019

Next up: checking if this actually runs in CI now

Copy link
Member

@gsnedders gsnedders left a comment

Choose a reason for hiding this comment

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

Need to get this running on CI, but otherwise yay!

tools/tox.ini Outdated Show resolved Hide resolved
tools/tox.ini Outdated Show resolved Hide resolved
tools/tox.ini Outdated Show resolved Hide resolved
@gsnedders
Copy link
Member

The CI failure here is just #16448.

@jgraham r?

@gsnedders gsnedders closed this Apr 23, 2019
@gsnedders gsnedders reopened this Apr 23, 2019
Copy link
Contributor

@jgraham jgraham left a comment

Choose a reason for hiding this comment

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

Oh this is ugly.

@gsnedders gsnedders merged commit 87f25e3 into master Apr 24, 2019
@gsnedders gsnedders deleted the mypy-manifest branch April 24, 2019 14:13
foolip added a commit that referenced this pull request May 17, 2021
This is faster locally when running mypy over and over, and shouldn't
matter in CI where there is no cache. The flag has been around ever
since mypy was introduced:
#16346

Mypy documentation of this flag:
https://mypy.readthedocs.io/en/stable/command_line.html#cmdoption-mypy-no-incremental
foolip added a commit that referenced this pull request May 18, 2021
This is faster locally when running mypy over and over, and shouldn't
matter in CI where there is no cache. The flag has been around ever
since mypy was introduced:
#16346

Mypy documentation of this flag:
https://mypy.readthedocs.io/en/stable/command_line.html#cmdoption-mypy-no-incremental
moz-v2v-gh pushed a commit to mozilla/gecko-dev that referenced this pull request May 20, 2021
…incremental flag), a=testonly

Automatic update from web-platform-tests
Enable mypy incremental mode (drop --no-incremental flag) (#29019)

This is faster locally when running mypy over and over, and shouldn't
matter in CI where there is no cache. The flag has been around ever
since mypy was introduced:
web-platform-tests/wpt#16346

Mypy documentation of this flag:
https://mypy.readthedocs.io/en/stable/command_line.html#cmdoption-mypy-no-incremental
--

wpt-commits: ffa4a71ed989ac3538b0bb44e632888e2b263d12
wpt-pr: 29019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants