type annotations cleanup

[eli-schwartz]

One minor fix for a broken annotation. Various stuff moved to typing-only imports.

[codecov]

Codecov Report

Merging #10248 (26fe41f) into master (5d0538d) will decrease coverage by 2.66%.
The diff coverage is n/a.

❗ Current head 26fe41f differs from pull request most recent head 1c52ac4. Consider uploading reports for the commit 1c52ac4 to get more accurate results

@@            Coverage Diff             @@
##           master   #10248      +/-   ##
==========================================
- Coverage   68.75%   66.08%   -2.67%     
==========================================
  Files         406      203     -203     
  Lines       87646    43813   -43833     
  Branches    19486     9575    -9911     
==========================================
- Hits        60260    28954   -31306     
+ Misses      22848    12606   -10242     
+ Partials     4538     2253    -2285     

Impacted Files	Coverage Δ
scripts/run_tool.py	0.00% <0.00%> (-100.00%)	⬇️
scripts/clangtidy.py	0.00% <0.00%> (-93.34%)	⬇️
modules/unstable_cuda.py	0.00% <0.00%> (-72.50%)	⬇️
templates/cudatemplates.py	37.50% <0.00%> (-62.50%)	⬇️
scripts/clangformat.py	0.00% <0.00%> (-54.84%)	⬇️
compilers/cuda.py	19.38% <0.00%> (-45.54%)	⬇️
dependencies/cuda.py	20.19% <0.00%> (-42.79%)	⬇️
compilers/d.py	37.23% <0.00%> (-19.28%)	⬇️
cmake/generator.py	48.14% <0.00%> (-11.12%)	⬇️
cmake/executor.py	58.08% <0.00%> (-6.59%)	⬇️
... and 252 more

---

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 5d0538d...1c52ac4. Read the comment docs.

[eli-schwartz]

@lazka seems like the msys2 CI has regressed.

[UNEXSKIP]   frameworks: 15 llvm    (method=cmake link-static=True)
 [SUCCESS]   frameworks: 15 llvm    (method=config-tool link-static=False)
 [SUCCESS]   frameworks: 15 llvm    (method=config-tool link-static=True)

[lazka]

We updated to llvm 14 just now. Hm, the logs aren't very helpful.

[lazka]

If I run this locally I get:

 [SKIPPED]   frameworks: 15 llvm    (method=cmake link-static=True)
Reason: llvm not found.
 [SUCCESS]   frameworks: 15 llvm    (method=config-tool link-static=False)
 [SUCCESS]   frameworks: 15 llvm    (method=config-tool link-static=True)

[lazka]

It's this I think: llvm/llvm-project#53950, so likely msys2 is the first llvm 14 user in CI?

[lazka]

I've filed #10249

[dcbaker]

Sorry it took me so long to review this, looks good to me.

[eli-schwartz]

Did a clean rebase to retrigger CI with the llvm 14 fix.

EDIT: Okay, in exchange we get a windows 2016 brownout lol.