Skip to content

Add optional delim argument#12

Merged
nlf merged 1 commit intoljharb:masterfrom
mjackson:opt-delim
Aug 7, 2014
Merged

Add optional delim argument#12
nlf merged 1 commit intoljharb:masterfrom
mjackson:opt-delim

Conversation

@mjackson
Copy link
Copy Markdown
Contributor

@mjackson mjackson commented Aug 7, 2014

This is useful for parsing/stringifying cookie values.

@mjackson
Add optional delim argument
a381a4b 
This is useful for parsing/stringifying cookie values.
nlf added a commit that referenced this pull request Aug 7, 2014
@nlf
Merge pull request #12 from mjackson/opt-delim
79072c4 
Add optional delim argument
@nlf nlf merged commit 79072c4 into ljharb:master Aug 7, 2014
@nlf nlf added the enhancement label Aug 7, 2014
@nlf nlf added this to the 1.1.1 milestone Aug 7, 2014
@nlf nlf self-assigned this Aug 7, 2014
nlf added a commit that referenced this pull request Aug 7, 2014
@nlf
cleanup for #12
0676904
@nlf
Copy link
Copy Markdown
Collaborator

nlf commented Aug 7, 2014

Thanks! Published in v1.2.0

@mjackson
Copy link
Copy Markdown
Contributor Author

mjackson commented Aug 7, 2014

@nlf So quick! :D You're welcome

mjackson added a commit to mjackson/qs that referenced this pull request Aug 25, 2014
@mjackson
Enable using a RegExp as delimiter
870577e 
This fixes a regression that was introduced in ljharb#16 which prevents
anything but a string from being used as a delimiter. But the whole
point of ljharb#12 was to allow users to specify a RegExp to use in
queryString.split(delimiter).
@mjackson mjackson mentioned this pull request Aug 25, 2014
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@nlf nlf

Labels

enhancement

Projects

None yet

Milestone

1.2.0

Development

Successfully merging this pull request may close these issues.

2 participants

@mjackson @nlf