##// END OF EJS Templates
Preserve unavailable_pythons list for post_pr_test script.
Thomas Kluyver -
Show More
@@ -3,10 +3,10 b''
3 3 """
4 4 from test_pr import load_results, post_logs, post_results_comment, print_results
5 5
6 num, results, pr = load_results()
6 num, results, pr, unavailable_pythons = load_results()
7 7 results_urls = post_logs(results)
8 print_results(pr, results_urls)
9 post_results_comment(pr, results_urls, num)
8 print_results(pr, results_urls, unavailable_pythons)
9 post_results_comment(pr, results_urls, num, unavailable_pythons)
10 10
11 11 print()
12 12 print("Posted test results to pull request")
@@ -128,7 +128,7 b' def run_tests(venv):'
128 128 # Restore $PATH
129 129 os.environ["PATH"] = orig_path
130 130
131 def markdown_format(pr, results_urls):
131 def markdown_format(pr, results_urls, unavailable_pythons):
132 132 def format_result(py, passed, gist_url, missing_libraries):
133 133 s = "* %s: " % py
134 134 if passed:
@@ -151,11 +151,11 b' def markdown_format(pr, results_urls):'
151 151 "Not available for testing: " + ", ".join(unavailable_pythons)]
152 152 return "\n".join(lines)
153 153
154 def post_results_comment(pr, results, num):
155 body = markdown_format(pr, results)
154 def post_results_comment(pr, results, num, unavailable_pythons=unavailable_pythons):
155 body = markdown_format(pr, results, unavailable_pythons)
156 156 gh_api.post_issue_comment(gh_project, num, body)
157 157
158 def print_results(pr, results_urls):
158 def print_results(pr, results_urls, unavailable_pythons=unavailable_pythons):
159 159 print("\n")
160 160 if pr['mergeable']:
161 161 print("**Test results for commit %s merged into master**" % pr['head']['sha'][:7])
@@ -174,7 +174,7 b' def print_results(pr, results_urls):'
174 174
175 175 def dump_results(num, results, pr):
176 176 with open(os.path.join(basedir, 'lastresults.pkl'), 'wb') as f:
177 pickle.dump((num, results, pr), f)
177 pickle.dump((num, results, pr, unavailable_pythons), f)
178 178
179 179 def load_results():
180 180 with open(os.path.join(basedir, 'lastresults.pkl'), 'rb') as f:
General Comments 0
You need to be logged in to leave comments. Login now