-
-
Notifications
You must be signed in to change notification settings - Fork 29
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
Invoke super with the valid args to avoid ENOENT #115
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This seems good independent from fixing the issue or not...
I'm working on creating a failing test to reproduce the reported issue (before the fix) and confirming the fix corrects it. |
27332d3
to
b2c8376
Compare
Failing tests added and submitted as a separate PR so folks could review the CI run in #118. This PR was then rebased on top of that to show that the issue is properly being fixed. |
🎉 |
Thanks!! |
@rwjblue any objection to backporting this to 3.x for the sake of ember-cli users pre 2.14? |
Seems good to me! |
Calling super with only the
relativePath
meant thatbroccoli-persistent-filter
's implementation ofgetDestinationPath
(which callsisDirectory
) could not properly no-op when the file has been removed.Fixes ember-cli/ember-cli#7347