-
-
Notifications
You must be signed in to change notification settings - Fork 12
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/client bugs #55
Fix/client bugs #55
Conversation
Signed-off-by: Vanessa Sochat <vsochat@stanford.edu>
Signed-off-by: Vanessa Sochat <vsochat@stanford.edu>
Signed-off-by: Vanessa Sochat <vsochat@stanford.edu>
Note: just changed so that if there is no runscript, it alerts the user and exits. |
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.
I have not reviewed the implementation but run
and exec
now work as expected, thanks a lot!!
okay cool! I tested most bits so I'm good to merge. Please open another issue if you run into any more, and always glad to help :) I'll release this on pypi in a minute and post the link here. |
This will fix two client bugs:
I've pushed the container
vanessa/scif:hw
to test this fix, here is an example of arguments being passed correctly:In addition, I've updated the date in the headers.
This will close #54 #53