Skip to content

added more tests to cover additional scenarios, simplified code change #2

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

Open
wants to merge 22 commits into
base: master
Choose a base branch
from

Conversation

mwillerich
Copy link

No description provided.

@mwillerich mwillerich changed the title added tests to cover scenarios where the copy option isn't set added more tests to cover additional scenarios, simplified code change May 16, 2014
Chris Wren and others added 16 commits June 13, 2014 08:06
Ignore directories when using file expansion
before: Fatal error: Destination for target %s is not a directory
now: Fatal error: Destination build/js/app.min.js for target jadeUsemin-js is not a directory
This reverts commit 26639bc, reversing
changes made to 9477210.
If you specify multiple file specs and the first one sets move to false, it affects all subsequent ones even if you want them to have move = true.
added copy option to README
added tests to cover scenarios where the copy option isn't set
before: Fatal error: Destination for target %s is not a directory
now: Fatal error: Destination build/js/app.min.js for target jadeUsemin-js is not a directory
This reverts commit 26639bc, reversing
changes made to 9477210.
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.

6 participants