Skip to content
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

feat: support {String} patterns #155

Merged
merged 2 commits into from
Oct 19, 2017

Conversation

cletusw
Copy link
Contributor

@cletusw cletusw commented Oct 10, 2017

Fixes #150

@@ -1053,6 +1053,25 @@ describe('apply function', () => {
});
});

describe('with simple string patterns', () => {
it('can move multiple files', (done) => {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Wording could be better if anyone has suggestions.

Copy link
Member

Choose a reason for hiding this comment

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

describe('Patterns - {String}', () => { 
   it('Multiple Files', () => {})
})

But 🐦 , all good

@cletusw
Copy link
Contributor Author

cletusw commented Oct 10, 2017

@michael-ciniawsky Please review, thanks!

Copy link
Member

@alexander-akait alexander-akait left a comment

Choose a reason for hiding this comment

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

Seems good

@michael-ciniawsky michael-ciniawsky changed the title feat: Support simple string patterns feat: support {String} patterns Oct 11, 2017
Copy link
Member

@michael-ciniawsky michael-ciniawsky left a comment

Choose a reason for hiding this comment

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

@cletusw thx

@@ -1053,6 +1053,25 @@ describe('apply function', () => {
});
});

describe('with simple string patterns', () => {
it('can move multiple files', (done) => {
Copy link
Member

Choose a reason for hiding this comment

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

describe('Patterns - {String}', () => { 
   it('Multiple Files', () => {})
})

But 🐦 , all good

@michael-ciniawsky
Copy link
Member

@cletusw Please update the docs (README) aswell :)

@cletusw
Copy link
Contributor Author

cletusw commented Oct 11, 2017

@michael-ciniawsky Okay, I've added the changes to the README to my string-patterns branch but for some reason the PR is not updating.

@cletusw
Copy link
Contributor Author

cletusw commented Oct 11, 2017

Never mind, it just updated. Weird.

Copy link
Member

@michael-ciniawsky michael-ciniawsky left a comment

Choose a reason for hiding this comment

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

@cletusw Thx

@michael-ciniawsky michael-ciniawsky merged commit b6c2e66 into webpack-contrib:master Oct 19, 2017
@cletusw cletusw deleted the string-patterns branch October 19, 2017 15:52
@michael-ciniawsky michael-ciniawsky removed this from the 4.3.0 milestone Nov 22, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants