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: Update pre-commit repo revisions #471

Merged
merged 1 commit into from Apr 30, 2024

Conversation

cdwilson
Copy link
Collaborator

I had some issues using pre-commit locally w/ Python 3.12, e.g.

ERROR: Failed building wheel for python-Levenshtein-wheels

Auto-upgrading the pre-commit dependencies (pre-commit autoupdate) to their latest versions fixed these issues.

I also had to upgrade the pre-commit docker image to kiwicom/pre-commit:3.6.0 due to Lucas-C/pre-commit-hooks#83

Update pre-commit dependencies to the latest revisions to fix errors
when using pre-commit with newer versions of Python (e.g. 3.12).

Signed-off-by: Chris Wilson <chris@cgnd.dev>
Copy link

Visit the preview URL for this PR (updated for commit 9a4d4bc):

https://golioth-firmware-sdk-doxygen-dev--pr471-cdwilson-updat-kldlwf9q.web.app

(expires Mon, 06 May 2024 16:07:08 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: a9993e61697a3983f3479e468bcb0b616f9a0578

Copy link

Code Coverage

Type Coverage
lines 55.6% (1035 of 1862 lines)
functions 57.7% (94 of 163 functions)

Copy link
Contributor

@sam-golioth sam-golioth left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @cdwilson! We can go ahead and merge this one, since it's not an SDK code change.

@cdwilson cdwilson merged commit 1e6062e into main Apr 30, 2024
40 of 42 checks passed
@cdwilson cdwilson deleted the cdwilson/update-pre-commit branch April 30, 2024 01:53
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.

None yet

2 participants