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
Do not delete the vendor directory #1326
Conversation
@reset can you take a look at this when you have a minute please? |
FileUtils.cp(source_file, "#{destination}/#{relative_path}") | ||
else | ||
raise RuntimeError, | ||
"Unknown file type: `#{File.ftype(source_file)}' at `#{source_file}'!" |
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.
We should probably say what we were doing when we encountered the bad file, "Syncing X to X failed:"
@sethvargo looks good just rebase and reformat to match the Berkshelf coding style |
a3279d3
to
99c83eb
Compare
@reset rebased and updated that error message. Are there any other style things you see that I need to cleanup? |
Do not delete the vendor directory
@sethvargo looks good |
This PR uses a file syncer to avoid deleting the vendor directory (previous behavior). This also updates and fixes the failing tests
/cc @reset @mikelococo