polygerrit-ui/app/elements/shared/gr-comment/gr-comment.ts (5 lines):
	- line 135: // TODO: Move this out of gr-comment. gr-comment should not have a comments
	- line 636: // TODO: This is causing a re-render of <gr-comment> on every key
	- line 882: // TODO: Move this out of gr-comment. gr-comment should not have a comments
	- line 1052: * TODO: Also move this into the comments-service.
	- line 1053: * TODO: Figure out a good reloading strategy when deleting was successful.


javatests/com/google/gerrit/acceptance/rest/project/GetBranchIT.java (3 lines):
	- line 365: // TODO: is this a bug?
	- line 390: // TODO: is this a bug?
	- line 492: // TODO: a user without the 'Access Database' capability cannot see the refs/meta/version ref


polygerrit-ui/app/elements/shared/gr-rest-api-interface/gr-rest-api-impl.ts (2 lines):
	- line 324: // TODO: Make the authService constructor parameter required when we have
	- line 2165: * and also apparently a number. TODO (beckysiegel) remove need for


javatests/com/google/gerrit/server/patch/IntraLineLoaderTest.java (2 lines):
	- line 88: // TODO: expected failure
	- line 103: // TODO: expected failure


javatests/com/google/gerrit/acceptance/api/revision/RevisionDiffIT.java (2 lines):
	- line 806: // TODO: Fix this issue. This test documents the current behavior and ensures that we at least
	- line 857: // TODO: Fix this issue. This test documents the current behavior and ensures that we at least


polygerrit-ui/app/types/diff.ts (2 lines):
	- line 80: // TODO: Are these really of type string? Not able to trigger them, but the
	- line 92: // TODO: Undocumented, but used in code.


polygerrit-ui/app/models/change/change-model.ts (2 lines):
	- line 110: // TODO: This goes together with `_patchRange.patchNum' being set to `edit`,
	- line 120: // TODO: Figure out how to best enforce immutability of all states. Use Immer?


polygerrit-ui/app/api/diff.ts (2 lines):
	- line 97: // TODO: Not documented.
	- line 209: // TODO: Missing documentation


polygerrit-ui/app/elements/change/gr-change-view/gr-change-view.ts (2 lines):
	- line 1883: // TODO: This goes together with `change.current_revision` being set, which
	- line 1982: // TODO: Fetch and process files.


javatests/com/google/gerrit/acceptance/api/accounts/AccountIT.java (2 lines):
	- line 1732: // TODO: Issue 10769: Adding an already existing key should not result in a notification email
	- line 1903: // TODO: Issue 10769: Adding an already existing key should not result in a notification email


javatests/com/google/gerrit/acceptance/rest/binding/PluginProvidedChildRestApiBindingsIT.java (2 lines):
	- line 85: * <p>TODO: This should actually just move into MyPluginHttpModule. However, that doesn't work
	- line 86: * currently. This TODO is for fixing this bug.


polygerrit-ui/app/embed/diff/gr-diff/gr-diff-utils.ts (2 lines):
	- line 123: // TODO: This type should be exposed to gr-diff clients in a separate type file.
	- line 126: // TODO: Also document the required HTML attributes that thread elements must


polygerrit-ui/app/elements/shared/gr-comment-thread/gr-comment-thread.ts (1 line):
	- line 145: // TODO: Is this attribute needed for querySelector() or css rules?


java/com/google/gerrit/httpd/restapi/RestApiServlet.java (1 line):
	- line 473: // TODO: Here and on other collection methods: There is a bug that binds child views


polygerrit-ui/app/elements/diff/gr-diff-host/gr-diff-host.ts (1 line):
	- line 1014: // TODO: Maybe just compute from patchRange.base on the fly?


java/com/google/gerrit/server/restapi/change/Submit.java (1 line):
	- line 201: // TODO Allow submitting non-current revision by changing the current.


polygerrit-ui/app/elements/core/gr-navigation/gr-navigation.ts (1 line):
	- line 328: // TODO: Refactor to return only GeneratedWebLink[]


polygerrit-ui/app/elements/plugins/gr-dom-hooks/gr-dom-hooks.ts (1 line):
	- line 36: // TODO: this still can not prevent if plugin has invalid char


polygerrit-ui/app/embed/diff/gr-diff-highlight/gr-annotation.ts (1 line):
	- line 233: // TODO (viktard): Polyfill Array.from for IE10.


polygerrit-ui/app/models/bulk-actions/bulk-actions-model.ts (1 line):
	- line 12: // TODO: consider keeping only changeId's as the object might become stale


polygerrit-ui/app/elements/shared/gr-download-commands/gr-download-commands.ts (1 line):
	- line 137: // TODO: maybe unify with strToClassName from dom-util


polygerrit-ui/app/embed/diff/gr-diff-builder/gr-diff-builder.ts (1 line):
	- line 50: * TODO: For human languages whose orthographies use combining marks, this


polygerrit-ui/app/styles/shared-styles.ts (1 line):
	- line 198: * TODO: Remove these rules and change (plugin) users to rely on


polygerrit-ui/app/elements/plugins/gr-endpoint-decorator/gr-endpoint-decorator.ts (1 line):
	- line 135: // TODO: this should be replaced by accessing the property directly


java/com/google/gerrit/server/notedb/ChangeNotes.java (1 line):
	- line 253: /* TODO: This is now unused in the Gerrit code-base, however it is kept in the code


polygerrit-ui/app/embed/diff/gr-diff-highlight/gr-diff-highlight.ts (1 line):
	- line 450: // TODO (viktard): Drop empty first and last lines from selection.


polygerrit-ui/app/elements/shared/gr-js-api-interface/gr-plugin-endpoints.ts (1 line):
	- line 119: // TODO: the logic below seems wrong when:


java/com/google/gerrit/server/permissions/RefControl.java (1 line):
	- line 631: // TODO This isn't an accurate test.


polygerrit-ui/app/elements/diff/gr-comment-api/gr-comment-api.ts (1 line):
	- line 359: // TODO: It probably makes more sense to set the patch_set in


polygerrit-ui/app/embed/gr-diff.ts (1 line):
	- line 35: // TODO (dmfilippov): find a better solution


polygerrit-ui/app/elements/shared/gr-js-api-interface/gr-plugin-rest-api.ts (1 line):
	- line 134: // TODO: We are soon getting rid of the `errFn` altogether. There are only


polygerrit-ui/app/elements/change/gr-file-list/gr-file-list_html.ts (1 line):
	- line 414: <!-- TODO: Remove data-url as it appears its not used -->


java/com/google/gerrit/server/query/change/ChangeData.java (1 line):
	- line 1273: // TODO: instantiating the notes is too much. We don't want to parse NoteDb, we just want the


java/com/google/gerrit/server/patch/DiffOperations.java (1 line):
	- line 37: *   <li>The Intra-line diffs for a single file path (TODO:ghareeb).


polygerrit-ui/app/scripts/util.ts (1 line):
	- line 22: // TODO (dmfilippov): Each function must be exported separately. According to


java/com/google/gerrit/server/git/validators/ValidationMessage.java (1 line):
	- line 51: // TODO: Remove and move callers to ValidationMessage(String message, Type type)


polygerrit-ui/app/elements/shared/gr-cursor-manager/gr-cursor-manager.ts (1 line):
	- line 62: * TODO (beckysiegel) figure out why it can be undefined


java/com/google/gerrit/server/restapi/change/RevertSubmission.java (1 line):
	- line 321: // TODO (paiking): As a future change, the revert should just be done directly on the


java/com/google/gerrit/server/restapi/project/ListProjects.java (1 line):
	- line 341: && isNullOrEmpty(matchSubstring) // TODO: see Issue 10446


java/com/google/gerrit/httpd/HttpServletResponseRecorder.java (1 line):
	- line 72: // TODO: Remove @SuppressWarnings and add @Override when Google upgrades


java/com/google/gerrit/index/query/TimestampRangePredicate.java (1 line):
	- line 22: // TODO: Migrate this to IntegerRangePredicate


polygerrit-ui/app/elements/shared/gr-list-view/gr-list-view.ts (1 line):
	- line 237: // TODO: fix offset (including itemsPerPage)


polygerrit-ui/app/utils/dom-util.ts (1 line):
	- line 48: // TODO: maybe should have a better name for this


java/com/google/gerrit/server/restapi/project/DeleteDashboard.java (1 line):
	- line 45: // TODO: Implement delete of dashboards by API.


javatests/com/google/gerrit/acceptance/ssh/SshCommandsIT.java (1 line):
	- line 43: // TODO: It would be better to dynamically generate these lists


java/com/google/gerrit/server/index/change/ChangeField.java (1 line):
	- line 116: // TODO: Rename LEGACY_ID to NUMERIC_ID


java/com/google/gerrit/httpd/DirectChangeByCommit.java (1 line):
	- line 48: // TODO Link to a specific patch set, if one matched.


java/com/google/gerrit/server/restapi/project/PutConfig.java (1 line):
	- line 207: // TODO check why we have this restriction


java/com/google/gerrit/server/restapi/project/SetDashboard.java (1 line):
	- line 43: // TODO: Implement update of dashboards by API.


java/com/google/gerrit/server/submit/MergeOp.java (1 line):
	- line 846: // TODO This is actually an error, the patch set ref exists but points to a revision that


polygerrit-ui/app/styles/themes/app-theme.ts (1 line):
	- line 220: /* TODO: Find a nicer way to combine the --assignee-highlight-color and the


java/com/google/gerrit/pgm/rules/PrologCompiler.java (1 line):
	- line 225: // TODO: fixify this loop


java/com/google/gerrit/server/change/ChangeInserter.java (1 line):
	- line 442: /* TODO: fixStatusToMerged is used here because the tests


polygerrit-ui/app/elements/core/gr-router/gr-router.ts (1 line):
	- line 1833: // TODO: Decouple the gr-app error view from network responses.


java/com/google/gerrit/server/git/meta/MetaDataUpdate.java (1 line):
	- line 114: *         // TODO handle exception


java/com/google/gerrit/server/change/ChangeKindCacheImpl.java (1 line):
	- line 416: // TODO - dborowitz: add NEW_CHANGE type for default.