iiasa / ixmp

The ix modeling platform for integrated and cross-cutting scenario analysis
https://docs.messageix.org/ixmp
Apache License 2.0
36 stars 110 forks source link

Use pre-commit to invoke ruff, mypy #492

Closed khaeru closed 9 months ago

khaeru commented 10 months ago

See iiasa/message_ix#742. This also includes the job step from iiasa/message-ix-models#124 / iiasa/message-ix-models#127.

Also:

How to review

Read the diff and note that the CI checks all pass.

PR checklist

codecov[bot] commented 10 months ago

Codecov Report

Merging #492 (3333827) into main (3a5479c) will decrease coverage by 0.1%. The diff coverage is 100.0%.

@@           Coverage Diff           @@
##            main    #492     +/-   ##
=======================================
- Coverage   98.5%   98.5%   -0.1%     
=======================================
  Files         42      42             
  Lines       4557    4554      -3     
=======================================
- Hits        4493    4489      -4     
- Misses        64      65      +1     
Files Changed Coverage Δ
ixmp/__init__.py 100.0% <100.0%> (ø)

... and 1 file with indirect coverage changes