Skip to content

array replacement #3

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

Closed
wants to merge 2 commits into from
Closed

array replacement #3

wants to merge 2 commits into from

Conversation

sashless
Copy link
Contributor

@sashless sashless commented Dec 4, 2015

Be able to use an array for replacements

for #1 (comment)

sashless and others added 2 commits November 13, 2015 17:03
can pass a complete array instead of just single values
can pass array as option to replace more at once
if (typeof query.search !== 'undefined' && typeof query.replace !== 'undefined') {
if (typeof query.flags !== 'undefined') {
query.search = new RegExp(query.search, query.flags);
}
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

readd regex + flags

@sashless
Copy link
Contributor Author

sashless commented Dec 4, 2015

alright i think we are done here ?

@sashless
Copy link
Contributor Author

sashless commented Dec 4, 2015

ah well need to push first ;D

@sashless
Copy link
Contributor Author

sashless commented Dec 4, 2015

i used master for the PR , my bad :/ closed in favor of https://github.com/Va1/string-replace-loader/pull/4/files @sergeylaptev

@sashless sashless closed this Dec 4, 2015
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.

2 participants