src/repository/api/ArcanistSubversionAPI.php (7 lines): - line 218: // TODO: This could be more accurate, but is only used by `arc browse` - line 296: // TODO: Provide a binary_safe_diff script for Windows. - line 297: // TODO: Provide a diff command which can take lines of context somehow. - line 339: // TODO: Hack for Windows. - line 401: // TODO: Move this to configuration? - line 526: // TODO: Handle paths with newlines. - line 539: // TODO: Handle paths with newlines. src/workflow/ArcanistDiffWorkflow.php (6 lines): - line 103: // TODO: Support mercurial. - line 750: // TODO: We should move this to a higher-level and put an indirection - line 1436: // TODO: We should offer to create a checkpoint commit. - line 2763: // TODO: This can be optimized if we're able to learn the "revisionPHID" - line 2903: // TODO: provide parent info - line 3445: // TODO: Eventually, we should expect these to succeed if we get this src/parser/ArcanistDiffParser.php (5 lines): - line 109: // TODO: In theory, you could have an '@' in your path and this could - line 608: // TODO: Build synthetic diffs for property changes, too. - line 706: // TODO: "hg diff -g" diffs ("mercurial git-style diffs") do not include - line 813: // TODO: We could decode the patches, but it's a giant mess so don't bother - line 1261: // TODO: Fix this. src/workflow/ArcanistLandWorkflow.php (5 lines): - line 513: // TODO: This message is misleading when landing a detached head or - line 1035: // TODO: Remove this once we're far enough in the future that - line 1639: // TODO: This is oh-so-gross because the below method is gross. - line 1649: // TODO: These queries should page through all results instead of fetching - line 1810: // TODO: This is oh-so-gross. src/workflow/ArcanistPatchWorkflow.php (5 lines): - line 677: // TODO: The SVN patch workflow likely does not work on windows because - line 879: // TODO Check if this works. - line 986: // TODO: See T848 for the authenticated stuff. - line 1041: // TODO: Pull open diffs from 'arc list'? - line 1204: // TODO -- de-hack this as permissions become more clear with things src/repository/api/ArcanistGitAPI.php (4 lines): - line 169: // TODO: Learn how cmd.exe works and find some clever workaround? - line 330: // TODO: Remove the history lesson soon. - line 456: // an alternative textual representation. TODO: Ideally, Differential - line 568: // TODO: Try to get something useful here. src/workflow/ArcanistWorkflow.php (4 lines): - line 766: // TODO: We'll always display these as long-form, when the user might - line 901: // TODO: This state can exist in Subversion, but it is currently handled - line 1117: // TODO: Check commits since tracking branch. If empty then return false. - line 1269: // TODO: Very gross src/unit/engine/ArcanistConfigurationDrivenUnitTestEngine.php (4 lines): - line 79: // TODO: Can we just do the type check first, and simplify this a bit? - line 139: // TODO: At some point, maybe we should emit a warning here if an engine - line 147: // TODO: Type check the results. - line 184: * TODO: This is copied from @{class:ArcanistConfigurationDrivenLintEngine}. src/workflow/ArcanistLintWorkflow.php (4 lines): - line 105: 'supports' => array('git', 'hg'), // TODO: svn - line 329: 'branch' => '', // TODO: Tracking branch. - line 514: // TODO: Improve the behavior here, make it more like - line 639: // TODO: Garbage collection. src/workflow/ArcanistStackWorkflow.php (4 lines): - line 409: // TODO: Clean this up and move it into LandEngines. - line 536: // TODO: This message is misleading when landing a detached head or - line 847: // TODO: This is oh-so-gross because the below method is gross. - line 852: // TODO: This is oh-so-gross. src/parser/ArcanistBundle.php (3 lines): - line 353: // TODO: We might be able to do this more cleanly after T230 is resolved. - line 387: // TODO: We should raise a FYI about this, so the user is aware - line 414: // TODO: This is only relevant when patching old Differential diffs src/land/ArcanistGitLandEngine.php (3 lines): - line 280: // TODO: This could probably be cleaner by asking the API a question - line 650: // TODO: Maybe this should also recover the proper upstream? - line 722: // TODO: This is working around Windows escaping problems, see T8298. src/lint/linter/xhpast/rules/ArcanistUndeclaredVariableXHPASTLinterRule.php (2 lines): - line 49: // TODO: Support functions defined inside other functions which is commonly - line 52: // TODO: parse_str() also makes lexical scope unknowable, see D13857. src/unit/engine/XUnitTestEngine.php (2 lines): - line 323: // FIXME: Can't use TempFile here as xUnit doesn't like - line 379: // FIXME: There's a bug in Mono which causes a segmentation fault src/unit/engine/PhpunitTestEngine.php (2 lines): - line 20: // TODO: add support for directories - line 118: * TODO: Add support for finding tests in testsuite folders from src/unit/engine/Phpunit6TestEngine.php (2 lines): - line 23: // TODO: add support for directories - line 121: * TODO: Add support for finding tests in testsuite folders from src/lint/engine/ArcanistLintEngine.php (2 lines): - line 258: // TODO: this may cause us to, e.g., load a large binary when we only - line 460: // TODO: The canRun() mechanism is only used by one linter, and just src/repository/api/ArcanistMercurialAPI.php (2 lines): - line 281: 'rev' => $node, // TODO: Remove eventually. - line 312: // TODO: Handle paths with newlines. src/workflow/ArcanistUploadWorkflow.php (2 lines): - line 84: // TODO: This could be handled more gracefully; just preserving behavior - line 165: // TODO: We could do these in parallel to improve upload performance. src/parser/diff/ArcanistDiffChange.php (1 line): - line 76: // TODO: The backend is shipping down some bogus data, e.g. diff 199453. src/lint/linter/xhpast/ArcanistXHPASTLinterRule.php (1 line): - line 145: * TODO: Improve this and move it to XHPAST proper? src/lint/linter/ArcanistSpellingLinter.php (1 line): - line 106: // TODO: This is a bit hacky. If no dictionaries were specified, then add src/unit/engine/UberMavenTestEngine.php (1 line): - line 113: // TODO: This will find *all* the pom.xml files in the working copy. src/workflow/ArcanistExportWorkflow.php (1 line): - line 219: // TODO: paths support src/workflow/ArcanistShellCompleteWorkflow.php (1 line): - line 181: // TODO: This is mega hacktown but something else probably breaks src/unit/engine/CSharpToolsTestEngine.php (1 line): - line 93: // FIXME: Can't use TempFile here as xUnit doesn't like src/lint/linter/xhpast/rules/ArcanistPHPCompatibilityXHPASTLinterRule.php (1 line): - line 220: // TODO: Technically, this will include function names. This is unlikely to src/lint/linter/xhpast/rules/ArcanistTodoCommentXHPASTLinterRule.php (1 line): - line 9: return pht('TODO Comment'); src/workflow/ArcanistUnitWorkflow.php (1 line): - line 411: // TODO: It would eventually be nice to stream test results up to the src/workflow/ArcanistLintersWorkflow.php (1 line): - line 181: // TODO: This should maybe use `tsprintf`. src/workflow/ArcanistDownloadWorkflow.php (1 line): - line 180: // TODO: We should stream responses to disk, but cURL gives us the raw src/configuration/ArcanistSettings.php (1 line): - line 459: // TODO: Both json_encode() and PhutilJSON do a bad job with one-liners. src/unit/engine/phutil/PhutilTestCase.php (1 line): - line 595: // TODO: This indicates "unreachable", but it flags the closing braces src/repository/api/ArcanistRepositoryAPI.php (1 line): - line 374: // TODO: Implement for Mercurial/SVN and make abstract. src/upload/ArcanistFileUploader.php (1 line): - line 261: // TODO: We could do these in parallel to improve upload performance. src/lint/linter/xhpast/rules/ArcanistClassExtendsObjectXHPASTLinterRule.php (1 line): - line 20: // TODO: This doesn't quite work for namespaced classes (see T8534). src/lint/linter/ArcanistJSHintLinter.php (1 line): - line 37: // TODO: If JSHint encounters a large number of errors, it will quit src/lint/linter/ArcanistCSharpLinter.php (1 line): - line 34: // TODO: This should probably be replaced with "bin" when this moves src/lint/linter/ArcanistXHPASTLinter.php (1 line): - line 127: // TODO: Improve this. src/workflow/ArcanistSubmitWorkflow.php (1 line): - line 129: // TODO: This message is misleading when landing a detached head or src/land/ArcanistLandEngine.php (1 line): - line 19: // TODO: This is really grotesque. scripts/arcanist.php (1 line): - line 265: // Set the URI path to '/api/'. TODO: Originally, I contemplated letting src/hgdaemon/ArcanistHgProxyClient.php (1 line): - line 126: // TODO: Do something with this? This is the 'debug' channel.