Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Merged
merged 4 commits into from
Sep 7, 2023
Merged

Conversation

@codecov
Copy link

codecov bot commented Apr 4, 2023

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.

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

@pre-commit-ci pre-commit-ci bot force-pushed the pre-commit-ci-update-config branch from db976a5 to 1134fbe Compare July 4, 2023 02:00
.pre-commit-config.yaml Outdated Show resolved Hide resolved
@webknjaz
Copy link
Contributor

webknjaz commented Sep 6, 2023

@samdoran #20 will fix the flake8 v6 crash seen in the pre-commit.ci report.

.pre-commit-config.yaml Outdated Show resolved Hide resolved
@webknjaz
Copy link
Contributor

webknjaz commented Sep 6, 2023

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

@samdoran
Copy link
Owner

samdoran commented Sep 6, 2023

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

@webknjaz
Copy link
Contributor

webknjaz commented Sep 6, 2023

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

@webknjaz
Copy link
Contributor

webknjaz commented Sep 6, 2023

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

@webknjaz
Copy link
Contributor

webknjaz commented Sep 6, 2023

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

@samdoran
Copy link
Owner

samdoran commented Sep 6, 2023

pre-commit.ci run

@samdoran
Copy link
Owner

samdoran commented Sep 6, 2023

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

@samdoran
Copy link
Owner

samdoran commented Sep 6, 2023

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

@samdoran
Copy link
Owner

samdoran commented Sep 6, 2023

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

@samdoran
Copy link
Owner

samdoran commented Sep 6, 2023

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
Copy link
Contributor

webknjaz commented Sep 7, 2023

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
Copy link
Contributor

webknjaz commented Sep 7, 2023

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.

.pre-commit-config.yaml Outdated Show resolved Hide resolved
pre-commit-ci bot and others added 3 commits September 7, 2023 12:40
updates:
- [github.com/asottile/add-trailing-comma.git: v2.4.0 → v3.0.0](https://github.com/asottile/add-trailing-comma.git/compare/v2.4.0...v3.0.0)
- [github.com/Lucas-C/pre-commit-hooks.git: v1.3.1 → v1.5.1](https://github.com/Lucas-C/pre-commit-hooks.git/compare/v1.3.1...v1.5.1)
- [github.com/adrienverge/yamllint.git: v1.28.0 → v1.32.0](https://github.com/adrienverge/yamllint.git/compare/v1.28.0...v1.32.0)
- [github.com/PyCQA/flake8.git: 5.0.4 → 6.0.0](https://github.com/PyCQA/flake8.git/compare/5.0.4...6.0.0)
- [github.com/PyCQA/flake8.git: 4.0.1 → 6.0.0](https://github.com/PyCQA/flake8.git/compare/4.0.1...6.0.0)
- [github.com/PyCQA/pylint.git: v2.15.9 → v3.0.0a6](https://github.com/PyCQA/pylint.git/compare/v2.15.9...v3.0.0a6)
Co-authored-by: Sviatoslav Sydorenko <[email protected]>
Co-authored-by: Sviatoslav Sydorenko <[email protected]>
v3.0.0 only supports Python >=3.6.
@samdoran samdoran merged commit 244ca72 into main Sep 7, 2023
17 checks passed
@samdoran samdoran deleted the pre-commit-ci-update-config branch September 7, 2023 17:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants