iza-institute-of-labor-economics / gettsim

The GErman Taxes and Transfers SIMulator
https://gettsim.readthedocs.io/
GNU Affero General Public License v3.0
54 stars 32 forks source link

[pre-commit.ci] pre-commit autoupdate #678

Closed pre-commit-ci[bot] closed 7 months ago

pre-commit-ci[bot] commented 9 months ago

updates:

codecov[bot] commented 9 months ago

Codecov Report

Attention: 1 lines in your changes are missing coverage. Please review.

Comparison is base (b324310) 0.00% compared to head (2ac410e) 91.54%.

Files Patch % Lines
src/_gettsim/visualization.py 0.00% 1 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #678 +/- ## ========================================= + Coverage 0 91.54% +91.54% ========================================= Files 0 51 +51 Lines 0 3443 +3443 ========================================= + Hits 0 3152 +3152 - Misses 0 291 +291 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

hmgaudecker commented 7 months ago

@lars-reimann, there were a couple of complaints by the hooks about the structure of the pyproject.toml file. While I did fix them, I also realised that setup.cfg can be completely replaced by pyproject.toml. Could you tackle this as part of this PR?

lars-reimann commented 7 months ago

@lars-reimann, there were a couple of complaints by the hooks about the structure of the pyproject.toml file. While I did fix them, I also realised that setup.cfg can be completely replaced by pyproject.toml. Could you tackle this as part of this PR?

Sure, I'll have look.

lars-reimann commented 7 months ago

@hmgaudecker Most entries are now moved from setup.cfg to pyproject.toml. I've added comments to the remaining ones explaining why they are not moved/don't need to be moved. If you are happy with them, setup.cfg can be removed completely (and setup-cfg-fmt can be disabled).