Fix bracket glob patterns

mstade commented 7 years ago

Description

As reported in #39, braced glob patterns don't work with ez-build, since it splits the patterns on , characters. This PR fixes that, so braced patterns work as well as others.

Motivation and Context

The ability to use braced glob patterns make some configuration much neater, with the canonical example being that which @DimitarChristoff mentioned in a comment to #37:

ez-build --include "js:**/*.{js,jsx}"

Compared to way described in the readme:

ez-build --include "js:**/*.js,js:**/*.jsx"

How Was This Tested?

[x] The opts parser unit tests were expanded with additional include/exclude pattern tests (source)
Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
Checklist:

[x] My change follows the style of this project
[x] My change requires a change to the documentation
[x] I have updated the documentation accordingly
[x] I have read the contribution guidelines
[x] I have added tests to cover my changes
[x] All new and existing tests passed

mstade commented 7 years ago

We should update the docs to use the neater braced syntax while we're at it.

codecov-io commented 7 years ago

Current coverage is 86.69% (diff: 100%)

Merging #41 into master will increase coverage by 0.10%

@@             master        #41   diff @@
==========================================
  Files            15         15          
  Lines           261        263     +2   
  Methods           0          0          
  Messages          0          0          
  Branches          0          0          
==========================================
+ Hits            226        228     +2   
  Misses           35         35          
  Partials          0          0

Powered by Codecov. Last update 71e04f4...c2c15fc

zambezi / ez-build