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

Cleanup of #3906 #7561

Merged
merged 3 commits into from
Jul 8, 2016
Merged

Cleanup of #3906 #7561

merged 3 commits into from
Jul 8, 2016

Conversation

jen20
Copy link
Contributor

@jen20 jen20 commented Jul 8, 2016

This pull-request builds upon the work in #3906 by cleaning up the temporary file created.

jorge.marey and others added 3 commits July 8, 2016 19:34
- Include new option in file provisioner. Now content or source can be
  provided. Content will create a temp file and copy there the contents.
- Later that file will be used as source.
- Include test to check that changes are working correctly.
@phinze
Copy link
Contributor

phinze commented Jul 8, 2016

LGTM pending travis

@jen20 jen20 merged commit 6084d73 into master Jul 8, 2016
@jen20 jen20 deleted the jorgemarey-f-file-content branch July 8, 2016 18:51
@ghost
Copy link

ghost commented Apr 24, 2020

I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues.

If you have found a problem that seems similar to this, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@ghost ghost locked and limited conversation to collaborators Apr 24, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants