[pre-commit.ci] pre-commit autoupdate

[pre-commit-ci[bot]]

updates:

• github.com/asottile/add-trailing-comma.git: v2.4.0 → v3.0.0
• github.com/Lucas-C/pre-commit-hooks.git: v1.3.1 → v1.5.1
• github.com/adrienverge/yamllint.git: v1.28.0 → v1.32.0
• github.com/PyCQA/flake8.git: 5.0.4 → 6.0.0
• github.com/PyCQA/flake8.git: 4.0.1 → 6.0.0
• github.com/PyCQA/pylint.git: v2.15.9 → v3.0.0a6

[codecov[bot]]

Codecov Report

Merging #19 (2844e7a) into main (762681d) will not change coverage. The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #19   +/-   ##
=======================================
  Coverage   35.49%   35.49%           
=======================================
  Files           5        5           
  Lines         386      386           
  Branches       48       48           
=======================================
  Hits          137      137           
  Misses        249      249           

Flag	Coverage Δ
sanity	32.34% <ø> (ø)
units	82.97% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

[webknjaz]

@samdoran https://github.com/samdoran/ansible-collection-macos/pull/20 will fix the flake8 v6 crash seen in the pre-commit.ci report.

[webknjaz]

@samdoran could you click "rebase" on the PR? I hope that with the updates to main, this might turn green already.

[samdoran]

I don't think a rebase is necessary since the checkout action usually does a speculative merge when testing.

[webknjaz]

A CI restart, then? It probably ran before those changes.

[webknjaz]

Though, you'd have to restart both GHA and pre-commit.ci which requires a specially crafted comment.

[webknjaz]

The latter can be achieved by posting the following comment: pre-commit.ci run

[samdoran]

pre-commit.ci run

[samdoran]

Not sure why it's not pulling in changes as expected. I'll just rebase this branch.

[samdoran]

Looks like the test failures are the same. I'll investigate.

[samdoran]

Looks like a few more import and compile ignores are needed due to the trailing comma change.

[samdoran]

Those trailing commas result in a lot of tests needing to be skipped. I wonder if we could just change the method signature instead.

I have a feeling once I get the import and compile tests passing, the tests themselves will fail when they try to run.

[webknjaz]

Not sure why it's not pulling in changes as expected. I'll just rebase this branch.

GitHub doesn't always update the ephemeral merge commit that many services use. It usually updates it when the PR is updated, not something external. Only automations that do this manually would work around this.

[webknjaz]

I have a feeling once I get the import and compile tests passing, the tests themselves will fail when they try to run.

Oh, right! I forgot that older Pythons didn't allow trailing commas after **kwargs.