geotribu / cli

Outil en ligne de commande pour gérer les tâches récurrentes du workflow Geotribu.
https://cli.geotribu.fr/
MIT License
2 stars 0 forks source link

refacto(header): transforme la liste de clés obligatoires en enum #225

Closed Guts closed 2 months ago

Guts commented 2 months ago

Pour info, je n'ai pas retiré "license" puisque c'est fait dans #215

sonarcloud[bot] commented 2 months ago

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud

codecov[bot] commented 2 months ago

Codecov Report

Attention: Patch coverage is 88.00000% with 3 lines in your changes missing coverage. Please review.

Project coverage is 67.12%. Comparing base (cd0c939) to head (892deba).

Additional details and impacted files [![Impacted file tree graph](https://app.codecov.io/gh/geotribu/cli/pull/225/graphs/tree.svg?width=650&height=150&src=pr&token=YRLQ6OPFRL&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=geotribu)](https://app.codecov.io/gh/geotribu/cli/pull/225?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=geotribu) ```diff @@ Coverage Diff @@ ## main #225 +/- ## ========================================== + Coverage 66.92% 67.12% +0.20% ========================================== Files 43 43 Lines 1938 1956 +18 Branches 343 347 +4 ========================================== + Hits 1297 1313 +16 - Misses 559 561 +2 Partials 82 82 ``` | [Flag](https://app.codecov.io/gh/geotribu/cli/pull/225/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=geotribu) | Coverage Δ | | |---|---|---| | [unittests](https://app.codecov.io/gh/geotribu/cli/pull/225/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=geotribu) | `66.87% <88.00%> (+0.20%)` | :arrow_up: | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=geotribu#carryforward-flags-in-the-pull-request-comment) to find out more. | [Files](https://app.codecov.io/gh/geotribu/cli/pull/225?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=geotribu) | Coverage Δ | | |---|---|---| | [geotribu\_cli/content/header\_check.py](https://app.codecov.io/gh/geotribu/cli/pull/225?src=pr&el=tree&filepath=geotribu_cli%2Fcontent%2Fheader_check.py&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=geotribu#diff-Z2VvdHJpYnVfY2xpL2NvbnRlbnQvaGVhZGVyX2NoZWNrLnB5) | `45.91% <83.33%> (-0.55%)` | :arrow_down: | | [geotribu\_cli/constants.py](https://app.codecov.io/gh/geotribu/cli/pull/225?src=pr&el=tree&filepath=geotribu_cli%2Fconstants.py&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=geotribu#diff-Z2VvdHJpYnVfY2xpL2NvbnN0YW50cy5weQ==) | `96.61% <89.47%> (-3.39%)` | :arrow_down: |
gounux commented 2 months ago

Pour info, je n'ai pas retiré "license" puisque c'est fait dans https://github.com/geotribu/cli/pull/215

À mes yeux c'est pas une mauvaise idée de le faire aussi ici, ça nous évitera quelques conflits ;)

Guts commented 2 months ago

Pour info, je n'ai pas retiré "license" puisque c'est fait dans #215

À mes yeux c'est pas une mauvaise idée de le faire aussi ici, ça nous évitera quelques conflits ;)

Ah aprdon, j'ai cliqué trop vite. Je m'occupe des conflits à la prochaine itération ;)