Skip to content

replace trim fn loop with regex#2085

Merged
gwyneplaine merged 1 commit intomasterfrom
update/replace-trim-fn
Oct 23, 2017
Merged

replace trim fn loop with regex#2085
gwyneplaine merged 1 commit intomasterfrom
update/replace-trim-fn

Conversation

@gwyneplaine
Copy link
Collaborator

@gwyneplaine gwyneplaine commented Oct 23, 2017

Minor adjustment to @serkanozer's PR #1862
Replaced the trim loop with a single regex statement.

@coveralls
Copy link

coveralls commented Oct 23, 2017

Coverage Status

Coverage decreased (-0.05%) to 92.879% when pulling 342136a on update/replace-trim-fn into c89e5b3 on master.

@gwyneplaine gwyneplaine merged commit 2275ada into master Oct 23, 2017
@gwyneplaine gwyneplaine deleted the update/replace-trim-fn branch October 23, 2017 03:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@gwyneplaine @coveralls