-
Notifications
You must be signed in to change notification settings - Fork 0
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
added cromwell tests docs in readme #34
Conversation
@seankross the failing test is unrelated since the only change here is in the readme. can you approve now? |
ping @seankross |
@sckott lemme run through this myself locally and then I will approve |
@seankross let me know if you run into any issues |
@sckott I just tried to run this and got this error. I likely need to investigate further, maybe my api key isn't right.
|
I tried this with |
@seankross can you try this:
I checked to make sure my env var of the same name was unset in my own dot files, and that the workflows started by the test suite were in fact in the test user account, which they were, so I think this is working "ON MY MACHINE" at least. lemme know howitdo |
That did it! Ugh I didn't read the docs closely enough. |
See #70 |
updated readme
woops, been a while since this was opened, and need to change some things, will send a new PR |
#33