Skip to content

Move Git config to git/, rename bin/ to config/, fix init/.gituser - #14

Merged
FrancesCoronel merged 7 commits into
masterfrom
claude/dotfiles-reorg
Oct 24, 2026
Merged

FrancesCoronel merged 7 commits into
masterfrom
claude/dotfiles-reorg

Conversation

@FrancesCoronel

Copy link
Copy Markdown
Owner

Before: the global Git config and ignore file lived in .github/, which reads like GitHub settings. Shell files, fonts and app themes lived in bin/, which held no executables. Option 3 of init/.gituser ("Copy over existing Git config") always failed, because $DOT was never set and the path pointed at bin/.github/.gitconfig.

After: Git files are in git/. Shell files, fonts and themes are in config/. .github/ holds only GitHub settings. Option 3 of init/.gituser copies git/.gitconfig to ~/.gitconfig.

How:

  • Moved .github/.gitconfig to git/.gitconfig.
  • Moved .github/.gitignore to git/gitignore_global. It was a dead ignore file, because a .gitignore inside .github/ only applies to that folder. The new name keeps it from acting as a repo ignore, and nothing installs it, same as before.
  • Renamed bin/ to config/, and updated every path in bootstrap.sh, init/.shell, init/.fonts, the README and AGENTS.md.
  • init/.gituser now sets DOT="$HOME/.dotfiles" like the other init scripts do.

Stacked on #13, because it updates the folder map that #13 adds to AGENTS.md. Frances approved this in the repo-structure audit (item 5).

Checks: ShellCheck passes on bootstrap.sh, init/.gituser, init/.fonts and init/.shell. A search finds no remaining bin/ or .github/.git* references. I didn't run the install scripts, per AGENTS.md.

🤖 Generated with Claude Code

https://claude.ai/code/session_012HeotRG6Zd1MKjxSnSmB6Z


Generated by Claude Code

Explains what each folder holds (including that .github/.gitconfig is
the global Git config and bin/ has no executables), warns agents never
to run bootstrap.sh or init/ scripts to test a change, and points
CLAUDE.md at it.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_012HeotRG6Zd1MKjxSnSmB6Z
…path

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_012HeotRG6Zd1MKjxSnSmB6Z
@FrancesCoronel FrancesCoronel self-assigned this Oct 13, 2026
Comment thread .github/workflows/pr.yml Fixed
Comment thread .github/workflows/pr.yml Fixed
Comment thread .github/workflows/pr.yml Fixed
FrancesCoronel and others added 3 commits October 15, 2026 09:54
CodeQL flags reusable workflows referenced by a mutable tag. Pin each to
b502794 (the current v1) with the tag kept as a comment for Dependabot.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Base automatically changed from claude/repo-structure to master October 24, 2026 14:25
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_012HeotRG6Zd1MKjxSnSmB6Z
@FrancesCoronel
FrancesCoronel marked this pull request as ready for review October 24, 2026 14:26
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 24, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ⚠️ Failed 2026-10-24T14:26:17.929838Z
We couldn’t complete this review. Comment @codex review to try again.
3a1eb73 Draft marked ready
ℹ️ 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" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@FrancesCoronel
FrancesCoronel merged commit 26ccb33 into master Oct 24, 2026
8 checks passed
@FrancesCoronel
FrancesCoronel deleted the claude/dotfiles-reorg branch October 24, 2026 14:29
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.

3 participants