-
Notifications
You must be signed in to change notification settings - Fork 539
feat: create Scm abstraction #2729
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Open
chingor13
wants to merge
37
commits into
main
Choose a base branch
from
local-github-scm
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from 33 commits
Commits
Show all changes
37 commits
Select commit
Hold shift + click to select a range
d8e6bc2
refactor: introduce scm abstraction
chingor13 131ccb9
feat(local-github): maintain temporary clone
chingor13 68f0602
feat(local-github): read files from local clone
chingor13 1bdf717
feat(local-github): find files by filename
chingor13 4f3f3e2
feat(local-github): find files by glob
chingor13 36cd502
feat(local-github): find files by extension
chingor13 5d172ea
feat(local-github): merge commit iterator
chingor13 f1c5192
refactor: common GitHub API tasks to delegate
chingor13 dbffbe3
feat(local-github): tag iterator
chingor13 e2fd590
feat(local-github): githubApiDelegate is required
chingor13 97ec586
chore: fix lint
chingor13 e860401
feat(cli): allow --local arg
chingor13 73a508d
feat(local-github): add logger
chingor13 4442475
fix(local-github): clone once during setup
chingor13 3819534
feat(local-github): support existing local clone
chingor13 150af21
feat(local-github): add --local-path CLI arg
chingor13 3accc78
refactor(local-github): git commands as stream
chingor13 0f3f1fb
fix: maxBuffer size and tree path
chingor13 9aef06f
fix: apply commits locally
chingor13 bc09783
test: fix snapshot
chingor13 8e50b15
fix: commit message file should not be committed
chingor13 e28b1b8
fix(local-github): update pull request directly uses API
chingor13 528552b
refactor: rename to GitHubApi
chingor13 4adfb49
refactor: move create options to GitHubApi
chingor13 a276678
refactor: move code-suggester code update logic to github class
chingor13 7d4fe01
feat!: remove unused createReleaasePullRequest method
chingor13 52e4f42
test: add local github tests
chingor13 c4e043e
fix: can read files from local clone on a different branch
chingor13 49b1602
test: ensure we can read files from a tag
chingor13 4cd1242
fix: fix finding files in other branches
chingor13 3fd9bdf
cleanup: add doc strings and removed unused functions
chingor13 d04eb70
chore: remove unused attribute
chingor13 0ff3d41
fix: use execFile
chingor13 408a741
fix: second order command injection
chingor13 c4bfff9
fix: clone repo if provided path is not a git clone
chingor13 988389b
feat(cli): add --local-clone-depth option
chingor13 d464e48
Merge branch 'main' into local-github-scm
chingor13 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe examples of how to use them?
For example, if
--localis true, are--local-pathand--local-clone-depthrequired? Or there are default values for them?