Mirror git-drs CLI to be like LFS#232
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 709b84ce76
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| replace github.com/calypr/data-client => ../data-client | ||
|
|
||
| replace github.com/calypr/syfon => ../syfon |
There was a problem hiding this comment.
Remove local path replaces from go.mod
Do not commit replace ... => ../... entries here: these paths only exist on the author's machine, so a normal clone cannot resolve github.com/calypr/data-client/syfon dependencies and go commands fail before tests or builds run. Keep these overrides in a local go.work or uncommitted dev-only config instead of repository go.mod.
Useful? React with 👍 / 👎.
| if len(branches) == 0 { | ||
| return lfs.GetWorktreeLfsFiles(logger) |
There was a problem hiding this comment.
Include hydrated LFS files in ls-files inventory
The default inventory path now calls GetWorktreeLfsFiles when branches is empty, but that function only returns files whose current worktree content is still an LFS pointer. After pull hydrates files, those tracked LFS files disappear from ls-files output, so status */localized reporting and DRS checks miss real tracked files.
Useful? React with 👍 / 👎.
|
TODO: address #29 |
addresses a bunch of issues see issues in PR linked.