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

Skip regex.sub in get_attr method when attribute is un-namespaced #710

Merged
merged 1 commit into from
Jun 19, 2024

Conversation

apasel422
Copy link
Contributor

On my local machine this reduces the execution time of se lint on Don Quixote from 55 seconds to 40 seconds.

Fixes #709

On my local machine this reduces the execution time of `se lint` on Don
Quixote from 55 seconds to 40 seconds.

Fixes standardebooks#709
@apasel422 apasel422 marked this pull request as ready for review June 18, 2024 21:46
@acabal acabal merged commit 6d8c352 into standardebooks:master Jun 19, 2024
1 check passed
@acabal
Copy link
Member

acabal commented Jun 19, 2024

Excellent, thanks!

@apasel422 apasel422 deleted the replace-speed branch June 19, 2024 17:15
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.

Consider removing _replace_shorthand_namespaces in favor of QName usage
2 participants