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.

Sign up for GitHub

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

Jump to bottom

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

Merged
merged 8 commits into from
Mar 18, 2024
Merged

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

merged 8 commits into from
Mar 18, 2024

Conversation

pre-commit-ci[bot]
Copy link
Contributor

@pre-commit-ci pre-commit-ci bot commented Jan 29, 2024 •
edited
Loading

@pre-commit-ci pre-commit-ci bot force-pushed the pre-commit-ci-update-config branch from a6fadc1 to 06fbcd7 Compare February 5, 2024 20:02
@pre-commit-ci pre-commit-ci bot force-pushed the pre-commit-ci-update-config branch from ecdea33 to 1e4257a Compare February 12, 2024 23:36
@pre-commit-ci pre-commit-ci bot force-pushed the pre-commit-ci-update-config branch from 5bfcf0d to 67720d7 Compare February 19, 2024 23:55
@scottbarnes
Copy link
Collaborator

scottbarnes commented Feb 20, 2024 •
edited
Loading

I will need someone else to review the changes I made to get this PR to pass all the tests.

Also, as a note to myself, my changes to this branch disappear every time pre-commit-ci pushes more changes, so I need to rebase off the branch and re-push every time that happens.

mekarpeles
mekarpeles approved these changes Feb 26, 2024
View reviewed changes
Copy link
Member

@mekarpeles mekarpeles left a comment

Choose a reason for hiding this comment

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

lgtm, pinging @hornc in case any of the marc code is still being used

@pre-commit-ci pre-commit-ci bot force-pushed the pre-commit-ci-update-config branch 2 times, most recently from fd07bd6 to eb92915 Compare March 18, 2024 21:40
pre-commit-ci bot and others added 7 commits March 18, 2024 15:28
@pre-commit-ci @scottbarnes
[pre-commit.ci] pre-commit autoupdate
f5be75b 
updates:
- [github.com/astral-sh/ruff-pre-commit: v0.1.14 → v0.2.2](astral-sh/ruff-pre-commit@v0.1.14...v0.2.2)
- [github.com/psf/black: 23.12.1 → 24.2.0](psf/black@23.12.1...24.2.0)
- [github.com/abravalheri/validate-pyproject: v0.15 → v0.16](abravalheri/validate-pyproject@v0.15...v0.16)
- [github.com/pre-commit/mirrors-eslint: v9.0.0-alpha.1 → v9.0.0-beta.0](pre-commit/mirrors-eslint@v9.0.0-alpha.1...v9.0.0-beta.0)
@pre-commit-ci @scottbarnes
[pre-commit.ci] auto fixes from pre-commit.com hooks
b386152 
for more information, see https://pre-commit.ci
@scottbarnes
marc/parse.py: remove unused code
02696f2 
This code for stripping a trailing `.` after three digits is unused.

The line before the removed code calls:
`pick_first_date()` -> `parse_date()` -> `remove_trailing_number_dot()`,
which calls a regex that will match anything the removed one does.

Removed regex:            `re_number_dot = re.compile(r'\d{3,}\.$')`
Regex that already runs`: `re_number_dot = re.compile(r'\d{2,}[- ]*(\.+)$')`
@scottbarnes
Remove unnecessary key check before dictionary access
3cd50dc
@scottbarnes
Avoid quadratic list summation
eec1926
@scottbarnes
Use dict.get('blah') instead of dict.get('blah', None)
a5c4edd
@scottbarnes
Add a test with four digits followed by a period
3499e8c
@scottbarnes scottbarnes force-pushed the pre-commit-ci-update-config branch from 1a3e78e to 3499e8c Compare March 18, 2024 22:29
@scottbarnes scottbarnes merged commit 3d1b5ae into master Mar 18, 2024
3 checks passed
rebecca-shoptaw pushed a commit to rebecca-shoptaw/openlibrary that referenced this pull request Mar 19, 2024
@pre-commit-ci @rebecca-shoptaw
[pre-commit.ci] pre-commit autoupdate (internetarchive#8773)
3a69634 
* marc/parse.py: remove unused code

This code for stripping a trailing `.` after three digits is unused.

* Remove unnecessary key check before dictionary access

* Avoid quadratic list summation

* Use `dict.get('blah')`  instead of `dict.get('blah', None)`

* Add a test with four digits followed by a period

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

---------
Co-authored-by: Scott Barnes <scottreidbarnes@gmail.com>
rebecca-shoptaw pushed a commit to rebecca-shoptaw/openlibrary that referenced this pull request Mar 19, 2024
@pre-commit-ci @rebecca-shoptaw
[pre-commit.ci] pre-commit autoupdate (internetarchive#8773)
67b930e 
* marc/parse.py: remove unused code

This code for stripping a trailing `.` after three digits is unused.

* Remove unnecessary key check before dictionary access

* Avoid quadratic list summation

* Use `dict.get('blah')`  instead of `dict.get('blah', None)`

* Add a test with four digits followed by a period

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

---------
Co-authored-by: Scott Barnes <scottreidbarnes@gmail.com>
merwhite11 pushed a commit to merwhite11/openlibrary that referenced this pull request Mar 27, 2024
@pre-commit-ci @merwhite11
[pre-commit.ci] pre-commit autoupdate (internetarchive#8773)
eb671c2 
* marc/parse.py: remove unused code

This code for stripping a trailing `.` after three digits is unused.

* Remove unnecessary key check before dictionary access

* Avoid quadratic list summation

* Use `dict.get('blah')`  instead of `dict.get('blah', None)`

* Add a test with four digits followed by a period

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

---------
Co-authored-by: Scott Barnes <scottreidbarnes@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@scottbarnes scottbarnes scottbarnes left review comments

@hornc hornc hornc left review comments

@mekarpeles mekarpeles mekarpeles approved these changes

Assignees

@scottbarnes scottbarnes

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@scottbarnes @hornc @mekarpeles