-
Notifications
You must be signed in to change notification settings - Fork 99
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
Add mounts
argument to more utils
components
#786
Comments
Sounds reasonable. Looking forward to the PR. Thanks! |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Description
The
python
,copy
, and maybe even thetouch
util components could benefit from having amounts
argument like thesh
util does.Motivation/Background
It's a slight convenience improvement to these utils because they might be used for dealing with e.g. mounted EFS volumes.
Detailed Proposal
Expand the signature of these util method definitions to include a
mounts
argument, and pass it into the correspondingRole
constructor.Alternatives
Wrap stuff in a call to
utils.sh
with your mounts.Additional context/links
Happy to contribute!
The text was updated successfully, but these errors were encountered: