Skip to content

Add Git Bare Namer #286

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.

Sign up for GitHub

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

Jump to bottom
Closed
wants to merge 26 commits into from
Closed

Add Git Bare Namer #286

wants to merge 26 commits into from
+165 −50

Conversation

joshmedeski
Copy link
Owner

@joshmedeski joshmedeski commented Sep 10, 2025 •
edited
Loading

In favor of #195

I simplified the detection of a git bare repo for the namer package.

  1. Call git worktree list and look for a (bare) entry
  2. Trim a /.bare suffix if it exists
  3. Use the path of the entry as the base for the name

jesseleite and others added 26 commits November 3, 2024 13:03
@jesseleite
Add CmdFromDir(), but maybe we could just accept another param in `…
d40ef9a 
…Cmd()`?
@jesseleite
Add GitMainWorktree(). (See git worktree —help for list command.)
d2e2cd2
@jesseleite
This currently gets root sesh name. Should it output path instead for…
e3f2f5a 
… fzf/gum?
@jesseleite
Output home shortened path to match Josh’s current output style for f…
ce9eefb 
…zf/gum.
@jesseleite
Remove CmdFromDir func in favor of git -C.
90914c8
@jesseleite
Update mocks.
bf4b4cc
@jesseleite
Add git worktree list test coverage.
a34c6bc
@jesseleite
This was never used/referenced anywhere. (And we’re about to refactor…
231f1a7 
… some of this anyway.)
@jesseleite
Refactor all .bare, —shot-top-level, —git-common-dir, etc. chec…
996c489 
…ks for more consistent `git worktree list` logic.
@jesseleite
Update mocks.
b048910
@jesseleite
Update test coverage.
420f9c0
@jesseleite
Simplify. This wrapper func seems redundant now.
b056333
@jesseleite
Make test example a bit more realistic.
d5fed1e
@jesseleite
Merge branch 'main' of github.com:joshmedeski/sesh into improved-sesh…
d73fcb6 
…-root-logic
@jesseleite
Add coverage for .bare directory convention.
ad58dd7
@jesseleite
Pass test for .bare support.
0a5c1c0
@jesseleite
Handle err.
c8657c2
@jesseleite
Handle empty fields output.
b746577
@jesseleite
Simplify git package to just run low level git commands.
659aa53
@jesseleite
Move worktree name parsing logic into namer, as requested.
9306b05
@jesseleite
Revert this stuff.
077a98c
@jesseleite
Add TODO.
c8e2aff
@jesseleite
Extract function and add test coverage.
f8cd3c5
@joshmedeski
Merge branch 'main' of github.com:joshmedeski/sesh into improved-sesh…
e95877b 
…-root-logic
@joshmedeski
Add common dir git func back
1e5920d
@joshmedeski
Restructure git bare convention to use (bare) from worktree list
e48840e 
- Break apart namer package into pieces
- Add logic to determine (bare) path name from worktree list command
- Add tests for determining logic
- Separate root function into separte source file
- Trim unused code
@joshmedeski
Copy link
Owner Author

I want to simplify the history and create a new PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@joshmedeski @jesseleite