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

Update mypy to 0.930 #105

Merged
merged 5 commits into from Dec 27, 2021
Merged

Update mypy to 0.930 #105

merged 5 commits into from Dec 27, 2021

Conversation

achimnol
Copy link
Member

This PR updates mypy to 0.930.

* While pyright (VSCode) supports ParamSpec and Concatenate,
  mypy 0.930 does not support Concatenate yet.
  Temporarily comment related lines with "type: ignore".
* Introduce a wrapper function to ensure immutable usage of
  ChainMap which is defined as mutable in typeshed, with minimal
  runtime overheads.
@achimnol achimnol added this to the 21.03 milestone Dec 27, 2021
@achimnol achimnol self-assigned this Dec 27, 2021
@codecov
Copy link

codecov bot commented Dec 27, 2021

Codecov Report

Merging #105 (66b5aeb) into main (686d6b7) will decrease coverage by 0.09%.
The diff coverage is 93.02%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #105      +/-   ##
==========================================
- Coverage   75.94%   75.85%   -0.10%     
==========================================
  Files          23       23              
  Lines        3201     3193       -8     
==========================================
- Hits         2431     2422       -9     
- Misses        770      771       +1     
Impacted Files Coverage Δ
src/ai/backend/common/etcd.py 82.67% <92.85%> (-0.82%) ⬇️
src/ai/backend/common/utils.py 58.79% <100.00%> (-0.57%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 686d6b7...66b5aeb. Read the comment docs.

@achimnol achimnol merged commit 9c4ee94 into main Dec 27, 2021
@achimnol achimnol deleted the fix/update-mypy-0.930 branch December 27, 2021 18:00
achimnol added a commit that referenced this pull request Dec 27, 2021
* setup: Update mypy to 0.930

* fix: Modernize common.etcd coding style and fix type errors

  - While pyright (VSCode) supports ParamSpec and Concatenate,
    mypy 0.930 does not support Concatenate yet.
    Temporarily comment related lines with "type: ignore".

  - Introduce a wrapper function to ensure immutable usage of
    ChainMap which is defined as mutable in typeshed, with minimal
    runtime overheads.

* test: Let test_utils type-checked and workaround python/mypy#11850

Backported-From: main (22.03)
Backported-To: 21.09
achimnol added a commit that referenced this pull request Dec 27, 2021
* setup: Update mypy to 0.930

* fix: Modernize common.etcd coding style and fix type errors

  - While pyright (VSCode) supports ParamSpec and Concatenate,
    mypy 0.930 does not support Concatenate yet.
    Temporarily comment related lines with "type: ignore".

  - Introduce a wrapper function to ensure immutable usage of
    ChainMap which is defined as mutable in typeshed, with minimal
    runtime overheads.

* test: Let test_utils type-checked and workaround python/mypy#11850

Backported-From: main (22.03)
Backported-To: 21.03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
1 participant