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

Fix SyntaxWarnings on regexes #3561

Closed
mlissner opened this issue Jan 5, 2024 · 2 comments
Closed

Fix SyntaxWarnings on regexes #3561

mlissner opened this issue Jan 5, 2024 · 2 comments
Assignees

Comments

@mlissner
Copy link
Member

mlissner commented Jan 5, 2024

Python 3.12 is revealing a few syntax warnings whenever we start things up:

/opt/courtlistener/cl/people_db/lookup_utils.py:331: SyntaxWarning: invalid escape sequence '\s'
  "(((Van|VAN|De|DE|Da|DA)\s)?[A-Z][\w\-'']{2,}(\s(IV|I|II|III|V|Jr\.|JR\.|Sr\.|SR\.))?),",
/usr/local/lib/python3.12/site-packages/judge_pics/search.py:37: SyntaxWarning: invalid escape sequence '\d'
  matching_path = re.sub("[\d-]+", " ", path).strip()

There's another for usaddress that I just filed a PR for, and another for scorched, which we will be getting rid of soon and can ignore, but the ones above are in our code and we should fix them.

@mlissner
Copy link
Member Author

Partially fixed here: freelawproject/judge-pics#23

@legaltextai legaltextai self-assigned this Aug 28, 2024
@s-taube s-taube moved this to General Backlog in Sprint (Web Team) Nov 25, 2024
@mlissner
Copy link
Member Author

And fully fixed in #4791.

@github-project-automation github-project-automation bot moved this from General Backlog to Done in Sprint (Web Team) Dec 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Status: CourtListener Backlog
Development

No branches or pull requests

2 participants