Closed bmcfee closed 2 months ago
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 88.32%. Comparing base (
7997fdf
) to head (b44cb8a
). Report is 5 commits behind head on main.
:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Pulling this change out of PR #377
The chord validator was compiling a monster regexp on each call, which is pretty wasteful. This PR just lifts that regexp precompilation up to the module so it can be reused. I belive this was the main inefficiency behind issue #341 .
There should be no functional changes here, so as long as it passes linting and CI I'm happy to merge.