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

This fixes (what I think) is a bug in copySync #52

Merged
merged 1 commit into from
Mar 1, 2014
Merged

This fixes (what I think) is a bug in copySync #52

merged 1 commit into from
Mar 1, 2014

Conversation

xogeny
Copy link

@xogeny xogeny commented Feb 26, 2014

The issue is that copySync applies the filter only at the root level of a recursive
copy. This adds a test (that failed with the original code) that checks to make sure that
the filter is applied recursively. The patch to lib/copy.js then addresses this issue
with a simple fix and the tests pass again with the patch.

The issue is that copySync applies the filter **only** at the root level of a recursive
copy.  This adds a test (that failed with the original code) that checks to make sure that
the filter is applied recursively.  The patch to lib/copy.js then addresses this issue
with a simple fix and the tests pass again with the patch.
mtiller added a commit to xogeny/cadeau that referenced this pull request Feb 26, 2014
This adds watching to all files that are copied from the users source
material (we don't watch the cadeau files).  However, this trick doesn't
quite work yet because of a bug in fs-extra which is detailed here:

jprichardson/node-fs-extra#52

But assuming that pull request is accepted, then once that fix is
available via npm then the watching should work fine.  Until then,
we wait.
jprichardson added a commit that referenced this pull request Mar 1, 2014
This fixes (what I think) is a bug in copySync
@jprichardson jprichardson merged commit 0fd7588 into jprichardson:master Mar 1, 2014
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