Skip to content

feat(gitter): importer endpoints#5690

Merged
Ly-Joey merged 13 commits into
google:masterfrom
Ly-Joey:feat-gitter-importer-endpoints
Jul 23, 2026
Merged

feat(gitter): importer endpoints#5690
Ly-Joey merged 13 commits into
google:masterfrom
Ly-Joey:feat-gitter-importer-endpoints

Conversation

@Ly-Joey

@Ly-Joey Ly-Joey commented Jul 20, 2026

Copy link
Copy Markdown
Contributor

Adds:

  • /file-diffs: List file diffs from last_synced_commit to HEAD of target branch (defaults to remote's origin/HEAD if not provided in request)
  • /file-content: Given commit hash and file path, git cat-file the content of the file

Also did some small refactoring to move code that handles repo fetching, cloning, etc into git.go.

There will be a follow-up PR to update importer to use these new endpoints.

@Ly-Joey
Ly-Joey force-pushed the feat-gitter-importer-endpoints branch from 5225fef to f92281d Compare July 20, 2026 01:44
Comment thread go/cmd/gitter/git.go Dismissed
Comment thread go/cmd/gitter/git.go Dismissed
Comment thread go/cmd/gitter/git.go Dismissed
Comment thread go/cmd/gitter/git.go Dismissed
Comment thread go/cmd/gitter/git.go Dismissed
@Ly-Joey
Ly-Joey marked this pull request as ready for review July 20, 2026 05:30
Comment thread go/cmd/gitter/gitter.go Outdated
Comment thread go/cmd/gitter/gitter.go Outdated
Comment thread go/cmd/gitter/gitter.go Outdated
Comment thread go/cmd/gitter/repository.go Outdated
Comment thread go/cmd/gitter/repository.go Outdated
Comment thread go/cmd/gitter/repository.go
Comment thread go/cmd/gitter/gitter.go Outdated
@Ly-Joey
Ly-Joey requested a review from another-rex July 23, 2026 05:09
michaelkedar
michaelkedar previously approved these changes Jul 23, 2026
Comment thread go/internal/gitter/pb/repository/repository.proto Outdated
@Ly-Joey
Ly-Joey merged commit d162c32 into google:master Jul 23, 2026
26 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants