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

Package updates #22

Merged
merged 2 commits into from
Jun 1, 2016
Merged

Package updates #22

merged 2 commits into from
Jun 1, 2016

Conversation

isaacs
Copy link
Contributor

@isaacs isaacs commented Dec 31, 2015

2 updates to package.json. One for license and the other to update glob and minimatch.

Note that as of glob 5, exclude patterns are supported out of the box. Maybe the dependency on minimatch isn't strictly necessary? It's hard to tell quickly.

Since glob 6 removes support for comment and negation patterns, this may or may not be a breaking change from fileset's pov.

Note that as of glob 5, exclude patterns are supported out of the box.
Maybe the dependency on minimatch isn't strictly necessary?  It's hard
to tell quickly.

Since glob 6 removes support for comment and negation patterns, this may
or may not be a breaking change from fileset's pov.
Over the last year or so, we've settled on `"license":"<SPDX-term>"` for
specifying licenses in a more machine-readable and human-convenient
format.
@riy
Copy link

riy commented Mar 2, 2016

It would be awesome if this pull request could be accepted by Mickael. The Glob version is hopelessly outdated.

@mklabs
Copy link
Owner

mklabs commented Jun 1, 2016

I dont know how I missed that PR.

Thanks a lot @isaacs and sorry for the delay. Merging.

Will follow up with a publish to the registry a bit later today.

@mklabs mklabs merged commit a75d21f into mklabs:master Jun 1, 2016
@mklabs
Copy link
Owner

mklabs commented Jun 1, 2016

Note that as of glob 5, exclude patterns are supported out of the box. Maybe the dependency on minimatch isn't strictly necessary? It's hard to tell quickly.

Yes, I recently saw that you added the ignore option, which is really really handy. It almost remove the need for a wrapper like fileset. Though, packages like istanbul depends on it, so hard to deprecate or introduce major breaking changes.

Thanks again for the PR, really appreciated.

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.

3 participants