Commit 995f87df authored by jeffcarp's avatar jeffcarp Committed by Commit bot

[WPT Export] Clean up TODOs

BUG=657117
R=qyearsley@chromium.org

Review-Url: https://codereview.chromium.org/2622033002
Cr-Commit-Position: refs/heads/master@{#442783}
parent c5f7441b
...@@ -4,11 +4,6 @@ ...@@ -4,11 +4,6 @@
"""A script for exporting and importing changes between the Chromium repo """A script for exporting and importing changes between the Chromium repo
and the web-platform-tests repo. and the web-platform-tests repo.
TODO(jeffcarp): does not handle reverted changes right now
TODO(jeffcarp): it also doesn't handle changes to -expected.html files
TODO(jeffcarp): Currently this script only does export; also add an option
import as well.
""" """
import argparse import argparse
......
...@@ -11,8 +11,6 @@ _log = logging.getLogger(__name__) ...@@ -11,8 +11,6 @@ _log = logging.getLogger(__name__)
CHROMIUM_WPT_DIR = 'third_party/WebKit/LayoutTests/imported/wpt/' CHROMIUM_WPT_DIR = 'third_party/WebKit/LayoutTests/imported/wpt/'
# TODO(jeffcarp): have the script running this fetch Chromium origin/master
# TODO(jeffcarp): move WPT fetch out of its constructor to match planned ChromiumWPT pattern
class TestExporter(object): class TestExporter(object):
......
...@@ -58,8 +58,6 @@ class WPTGitHub(object): ...@@ -58,8 +58,6 @@ class WPTGitHub(object):
assert desc_title assert desc_title
assert body assert body
# TODO(jeffcarp): CC foolip and qyearsley on all PRs for now
# TODO(jeffcarp): add HTTP to Host and use that here
path = '/repos/w3c/web-platform-tests/pulls' path = '/repos/w3c/web-platform-tests/pulls'
body = { body = {
"title": desc_title, "title": desc_title,
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment