internal/command/command.go (2 lines): - line 408: //nolint:errcheck // TODO: do we want to report errors? - line 522: // TODO: use errors.Join() to combine errors instead proto/go/gitalypb/commit.pb.go (1 line): - line 84: // TODO: Replace this enum with ObjectType in shared.proto internal/git/packfile/index.go (1 line): - line 104: // TODO use a data structure other than a Go slice to hold the index internal/cli/gitaly/serve.go (1 line): - line 727: // TODO should this be a return? https://gitlab.com/gitlab-org/gitaly/issues/1893 internal/git/localrepo/merge.go (1 line): - line 270: // TODO: for now, it's better that this error matches the git2go internal/gitaly/service/operations/user_create_branch.go (1 line): - line 42: // BEGIN TODO: Uncomment if StartPoint started behaving sensibly internal/gitaly/repoutil/create.go (1 line): - line 182: // TODO: Ideally we want to also use the same ideology for the files backend too internal/praefect/testserver.go (1 line): - line 234: // TODO: run a replmgr for EVERY virtual storage cmd/gitaly-ssh/main.go (1 line): - line 52: // TODO: Errors needs to be sent back some other way... pipes? internal/git/reftable.go (1 line): - line 332: // TODO: CRC32 validation of the data internal/git/lfs.go (1 line): - line 15: // TODO: this is incomplete as it does not recognize pre-release version of LFS blobs with internal/gitaly/service/operations/submodules.go (1 line): - line 130: // TODO: this needs to be converted to a structured error, and once done we should stop proto/commit.proto (1 line): - line 353: // TODO: Replace this enum with ObjectType in shared.proto internal/praefect/middleware/errorhandler.go (1 line): - line 21: // TODO: use errors.Join() to combine errors instead internal/gitaly/hook/referencetransaction.go (1 line): - line 187: // TODO: This can be removed once the bug in Git itself is fixed internal/gitaly/service/objectpool/delete.go (1 line): - line 19: // TODO: we really should return an error in case we're trying to delete an internal/git/catfile/parse_commit.go (1 line): - line 126: // TODO: Current tests allow empty commits, we might want internal/testhelper/testdb/db.go (1 line): - line 191: // TODO: In Postgre 14, pg_terminate_backend takes an optional timeout argument that makes it a blocking internal/git/localrepo/tag.go (1 line): - line 80: // TODO: Upper-case error message purely for transitory backwards compatibility internal/gitaly/service/repository/repository_exists.go (1 line): - line 19: // TODO: this error case should really be converted to an actual error. internal/git/reference.go (1 line): - line 162: // TODO: this can eventually be converted to use `strings.CutPrefix()`. internal/backup/backup.go (1 line): - line 437: // TODO: Remove refs check after couple of releases.