Skip to content

jj: use bedag work identity in bedag repos - #21

Merged
phonkd merged 1 commit into
mainfrom
worktree-jj-homemanager-config
Jul 9, 2026
Merged

jj: use bedag work identity in bedag repos#21
phonkd merged 1 commit into
mainfrom
worktree-jj-homemanager-config

Conversation

@phonkd

@phonkd phonkd commented Jul 9, 2026

Copy link
Copy Markdown
Owner

Summary

  • Adds home-manager/jjconfig.nix in bedag-setup, mirroring gitconfig.nix's pattern: instead of git's includeIf/hasconfig: (URL-based), jj's conditional config uses --when.repositories (path-based) since jj has no remote-URL condition. Scopes user.name/user.email to the bedag identity (config.bedag.name/config.bedag.mail) under ~/git/bedag and ~/git/bedag-setup.
  • Wires it into modules/work/external.nix next to the existing gitconfig.nix import.

Test plan

  • nix-instantiate --parse on both changed files
  • home-manager switch on the work host, then jj log inside a ~/git/bedag/* repo to confirm Author shows the bedag identity, and outside it shows the personal one

Adds home-manager/jjconfig.nix in bedag-setup (mirroring gitconfig.nix's
conditional includes) and imports it, so jj uses the bedag work identity
under ~/git/bedag and ~/git/bedag-setup instead of the personal one.
@phonkd
phonkd marked this pull request as ready for review July 9, 2026 08:20
Copilot AI review requested due to automatic review settings July 9, 2026 08:20
@phonkd
phonkd merged commit fdac950 into main Jul 9, 2026
2 checks passed

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR wires the new jjconfig.nix from the external bedag-setup Home Manager config into the existing work-external-config module, so jj can pick up bedag-scoped identity settings when that external repo is present.

Changes:

  • Add jjconfig.nix to the imports list sourced from ~/git/bedag-setup/home-manager.
  • Keeps the existing “bedag-setup as an external Home Manager module set” wiring pattern intact.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@phonkd
phonkd deleted the worktree-jj-homemanager-config branch July 9, 2026 16:46
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.

2 participants