search

tianocore / edk2-pytool-extensions

Extensions to the edk2 build system allowing for a more robust and plugin based build system and tool execution environment
Other
60 stars 41 forks source link

[REBASE&FF] Ruff on workspace #892

Closed Javagedes closed 1 month ago

Javagedes commented 1 month ago

pyproject.toml: Update ruff deprecated settings

Addresses these deprecated messages from ruff:

warning: The top-level linter settings are deprecated in favour of
their counterparts in the `lint` section. Please update the
following options in `pyproject.toml`:

  - 'ignore' -> 'lint.ignore'
  - 'select' -> 'lint.select'
  - 'flake8-annotations' -> 'lint.flake8-annotations'
  - 'pydocstyle' -> 'lint.pydocstyle'

Fix ruff issues

Runs ruff format . on the workspace.

codecov[bot] commented 1 month ago

Codecov Report

Attention: Patch coverage is 79.59442% with 161 lines in your changes missing coverage. Please review.

Project coverage is 78.81%. Comparing base (bf643b0) to head (fa4afe8). Report is 111 commits behind head on master.

Files with missing lines Patch % Lines
edk2toolext/capsule/signtool_signer.py 55.55% 20 Missing :warning:
edk2toolext/environment/uefi_build.py 73.97% 19 Missing :warning:
edk2toolext/versioninfo/versioninfo_helper.py 67.56% 12 Missing :warning:
edk2toolext/invocables/edk2_ci_build.py 28.57% 10 Missing :warning:
edk2toolext/nuget_publishing.py 89.02% 9 Missing :warning:
edk2toolext/capsule/capsule_tool.py 63.63% 8 Missing :warning:
...ronment/extdeptypes/az_cli_universal_dependency.py 27.27% 8 Missing :warning:
edk2toolext/omnicache.py 91.01% 8 Missing :warning:
edk2toolext/versioninfo/versioninfo_tool.py 22.22% 7 Missing :warning:
edk2toolext/capsule/pyopenssl_signer.py 70.00% 6 Missing :warning:
... and 16 more
Additional details and impacted files [![Impacted file tree graph](https://app.codecov.io/gh/tianocore/edk2-pytool-extensions/pull/892/graphs/tree.svg?width=650&height=150&src=pr&token=vVJxZexcTI&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=tianocore)](https://app.codecov.io/gh/tianocore/edk2-pytool-extensions/pull/892?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=tianocore) ```diff @@ Coverage Diff @@ ## master #892 +/- ## ========================================== + Coverage 78.75% 78.81% +0.05% ========================================== Files 49 41 -8 Lines 4909 5050 +141 ========================================== + Hits 3866 3980 +114 - Misses 1043 1070 +27 ``` | [Files with missing lines](https://app.codecov.io/gh/tianocore/edk2-pytool-extensions/pull/892?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=tianocore) | Coverage Δ | | |---|---|---| | [edk2toolext/bin/nuget.py](https://app.codecov.io/gh/tianocore/edk2-pytool-extensions/pull/892?src=pr&el=tree&filepath=edk2toolext%2Fbin%2Fnuget.py&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=tianocore#diff-ZWRrMnRvb2xleHQvYmluL251Z2V0LnB5) | `89.28% <100.00%> (-0.37%)` | :arrow_down: | | [edk2toolext/codeql.py](https://app.codecov.io/gh/tianocore/edk2-pytool-extensions/pull/892?src=pr&el=tree&filepath=edk2toolext%2Fcodeql.py&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=tianocore#diff-ZWRrMnRvb2xleHQvY29kZXFsLnB5) | `100.00% <100.00%> (ø)` | | | [edk2toolext/edk2\_logging.py](https://app.codecov.io/gh/tianocore/edk2-pytool-extensions/pull/892?src=pr&el=tree&filepath=edk2toolext%2Fedk2_logging.py&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=tianocore#diff-ZWRrMnRvb2xleHQvZWRrMl9sb2dnaW5nLnB5) | `88.46% <100.00%> (-0.07%)` | :arrow_down: | | [edk2toolext/environment/conf\_mgmt.py](https://app.codecov.io/gh/tianocore/edk2-pytool-extensions/pull/892?src=pr&el=tree&filepath=edk2toolext%2Fenvironment%2Fconf_mgmt.py&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=tianocore#diff-ZWRrMnRvb2xleHQvZW52aXJvbm1lbnQvY29uZl9tZ210LnB5) | `100.00% <100.00%> (ø)` | | | [...2toolext/environment/extdeptypes/git\_dependency.py](https://app.codecov.io/gh/tianocore/edk2-pytool-extensions/pull/892?src=pr&el=tree&filepath=edk2toolext%2Fenvironment%2Fextdeptypes%2Fgit_dependency.py&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=tianocore#diff-ZWRrMnRvb2xleHQvZW52aXJvbm1lbnQvZXh0ZGVwdHlwZXMvZ2l0X2RlcGVuZGVuY3kucHk=) | `98.27% <100.00%> (-0.03%)` | :arrow_down: | | [...toolext/environment/plugintypes/ci\_build\_plugin.py](https://app.codecov.io/gh/tianocore/edk2-pytool-extensions/pull/892?src=pr&el=tree&filepath=edk2toolext%2Fenvironment%2Fplugintypes%2Fci_build_plugin.py&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=tianocore#diff-ZWRrMnRvb2xleHQvZW52aXJvbm1lbnQvcGx1Z2ludHlwZXMvY2lfYnVpbGRfcGx1Z2luLnB5) | `100.00% <100.00%> (ø)` | | | [...olext/environment/plugintypes/uefi\_build\_plugin.py](https://app.codecov.io/gh/tianocore/edk2-pytool-extensions/pull/892?src=pr&el=tree&filepath=edk2toolext%2Fenvironment%2Fplugintypes%2Fuefi_build_plugin.py&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=tianocore#diff-ZWRrMnRvb2xleHQvZW52aXJvbm1lbnQvcGx1Z2ludHlwZXMvdWVmaV9idWlsZF9wbHVnaW4ucHk=) | `60.00% <100.00%> (-6.67%)` | :arrow_down: | | [edk2toolext/environment/rust.py](https://app.codecov.io/gh/tianocore/edk2-pytool-extensions/pull/892?src=pr&el=tree&filepath=edk2toolext%2Fenvironment%2Frust.py&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=tianocore#diff-ZWRrMnRvb2xleHQvZW52aXJvbm1lbnQvcnVzdC5weQ==) | `86.56% <100.00%> (ø)` | | | [...toolext/environment/self\_describing\_environment.py](https://app.codecov.io/gh/tianocore/edk2-pytool-extensions/pull/892?src=pr&el=tree&filepath=edk2toolext%2Fenvironment%2Fself_describing_environment.py&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=tianocore#diff-ZWRrMnRvb2xleHQvZW52aXJvbm1lbnQvc2VsZl9kZXNjcmliaW5nX2Vudmlyb25tZW50LnB5) | `89.07% <100.00%> (+0.37%)` | :arrow_up: | | [edk2toolext/environment/shell\_environment.py](https://app.codecov.io/gh/tianocore/edk2-pytool-extensions/pull/892?src=pr&el=tree&filepath=edk2toolext%2Fenvironment%2Fshell_environment.py&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=tianocore#diff-ZWRrMnRvb2xleHQvZW52aXJvbm1lbnQvc2hlbGxfZW52aXJvbm1lbnQucHk=) | `91.61% <100.00%> (-0.06%)` | :arrow_down: | | ... and [31 more](https://app.codecov.io/gh/tianocore/edk2-pytool-extensions/pull/892?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=tianocore) | | ... and [9 files with indirect coverage changes](https://app.codecov.io/gh/tianocore/edk2-pytool-extensions/pull/892/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=tianocore)