-
Notifications
You must be signed in to change notification settings - Fork 9
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
Pythonize run_unit_tests.sh #58
base: master
Are you sure you want to change the base?
Pythonize run_unit_tests.sh #58
Conversation
This helper script for Seattle Testbed, https://seattle.poly.edu/, * downloads Seattle Testbed's sources from GitHub, * builds them using Seattle's build scripts, and finally * runs Seattle's unit tests.
This is part of the PR's test result:
|
Okay, thanks for your initiative on this. Here's a few things I noticed:
Did you have a chance to run this on Windows btw? Do we have a tester with access to a Win box? |
# For every repo name, ... | ||
for repo in repolist: | ||
print "Testing " + testbed + '/' + repo + " now..." | ||
print "https://github.com/" + testbed + "/" + repo + ".git" |
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.
Should we make the domain name configurable, too? For example, if a local or non-github git repository is needed.
This helper script for Seattle Testbed, https://seattle.poly.edu/,