Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1525)

Unified Diff: third_party/WebKit/Tools/Scripts/webkitpy/w3c/sync_wpt.py

Issue 2518313003: Refactor WPT Export to ensure only one PR in flight at a time (Closed)
Patch Set: Delete two unrelated files Created 4 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/Tools/Scripts/webkitpy/w3c/sync_wpt.py
diff --git a/third_party/WebKit/Tools/Scripts/webkitpy/w3c/sync_wpt.py b/third_party/WebKit/Tools/Scripts/webkitpy/w3c/sync_wpt.py
index f7e4d4066f3cea76d28ce884084090190a26df55..e326882dab2d264f3043ae88ec2ec75027dc3b4a 100644
--- a/third_party/WebKit/Tools/Scripts/webkitpy/w3c/sync_wpt.py
+++ b/third_party/WebKit/Tools/Scripts/webkitpy/w3c/sync_wpt.py
@@ -52,6 +52,13 @@ def main():
_log.info('No exportable commits found in Chromium, stopping.')
return
+ if options.status:
+ return
+
+ # should no longer iterate through all of them
+ # should at most one of these per process:
+ # - create a PR
+ # - check a PR status (and merge if green)
for commit in exportable_commits:
_log.info('Uploading %s', chromium_wpt.subject(commit))
chromium_commit = ChromiumCommit(host, sha=commit)
@@ -70,4 +77,5 @@ def main():
def parse_args():
parser = argparse.ArgumentParser(description='WPT Sync')
parser.add_argument('--no-fetch', action='store_true')
+ parser.add_argument('--status', action='store_true')
return parser.parse_args()

Powered by Google App Engine
This is Rietveld 408576698