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
Fix dependencies check for compile command #455
Conversation
rescue DependencyError | ||
false | ||
end | ||
|
||
def clean_ember_dependencies! | ||
ember_dependency_directories.select(&:exist?).each(&:rmtree) | ||
ember_dependency_directories.select(&:exist?).flat_map(&:children).each(&:rmtree) |
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.
Line is too long. [87/80]
It does not seem like specs fail because of my change. @seanpdoyle is there anything else I could do in order to get this merged? |
@krasnoukhov unfortunately, it looks like I'll try to dedicate some time soon to get I'll keep you posted! |
@seanpdoyle I went ahead and added a PR for specs fixing: #456 |
This [commit] changed return value of `run` command to be `process.value` instead of `status`, so we need to use `success?` now. [commit]: ba83120
Remove contents only instead. This helps when either of `node_modules` or `bower_components` is a symlink.
@seanpdoyle This is green now. Can you cut a bugfix release once you merge it? Thanks |
Merged in https://github.com/thoughtbot/ember-cli-rails/releases/tag/v0.7.4. Released in Thanks! |
Thanks 💚 |
First commit fixes actual bug, while the second one improves
clean_ember_dependencies!
behavior when dependencies directory is a symlink.We use shared directories for
node_modules
andbower_components
in order to speed up deploys. This change make cleanup work for that case.Thanks!