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

refactor: refactoring matchers.RepeatMatcher #137

Merged

vvatanabe merged 2 commits into master from refaactor-RepeatMatcher

Aug 8, 2023

Member

vvatanabe commented Aug 8, 2023

refactoring matchers.RepeatMatcher:

Extracted part of the process within the execute function into a separate function.
Reused the regular expression's Pattern object.
Applied google-java-format.
Reduced the calls to the group function in regex.Matcher.

vvatanabe force-pushed the refaactor-RepeatMatcher branch from f73b0a4 to 8c60f39 Compare

August 8, 2023 21:49

coveralls commented Aug 8, 2023 •

edited

coverage: 92.353% (-0.02%) from 92.369% when pulling 6d9b459 on refaactor-RepeatMatcher into b7f5840 on master.


          style: format code and optimize imports in matchers.RepeatMatcher

5a9526b

vvatanabe force-pushed the refaactor-RepeatMatcher branch from 3c1dd36 to 5a9526b Compare

August 8, 2023 21:52


          refactor: refactoring matchers.RepeatMatcher

6d9b459

vvatanabe merged commit 181cb40 into master

1 of 3 checks passed

vvatanabe deleted the refaactor-RepeatMatcher branch

August 15, 2023 07:17

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment