diff --git a/infra/bots/recipes/bookmaker.py b/infra/bots/recipes/bookmaker.py index 34f34948c9..62d53dd241 100644 --- a/infra/bots/recipes/bookmaker.py +++ b/infra/bots/recipes/bookmaker.py @@ -79,6 +79,10 @@ def RunSteps(api): content = api.file.read_text('Read fiddleout.json', fiddlecli_output, test_data=test_data) out = json.loads(content) + # Do a dump of fiddlecli_output. Will be useful for debugging. + print 'Dump of %s:' % fiddlecli_output + print json.dumps(out, indent=4) + failing_fiddles = [] for fiddle_name in out: props = out[fiddle_name] @@ -91,10 +95,6 @@ def RunSteps(api): failure_msg += 'https://fiddle.skia.org/c/%s\n' % fiddle_hash raise api.step.StepFailure(failure_msg) - # Do a dump of fiddlecli_output. Will be useful for debugging. - print 'Dump of %s:' % fiddlecli_output - print json.dumps(out, indent=4) - # Step 4: Update docs in site/user/api/ with the output of fiddlecli. # If there are any new changes then upload and commit the changes. update_docs_gitcookies = api.path['start_dir'].join( diff --git a/infra/bots/upload_md.py b/infra/bots/upload_md.py index 6ee5ba13a7..fee4a1b677 100644 --- a/infra/bots/upload_md.py +++ b/infra/bots/upload_md.py @@ -74,7 +74,7 @@ def main(): sys.exit(1) if diff_files: subprocess.check_call(['git', 'add', '-u']) - git_branch.commit_and_upload(True) + git_branch.commit_and_upload(False) else: print 'No changes so nothing to upload.'