feat(POLISH-004): add .editorconfig + .inputrc cross-OS polish (#133)#178
Merged
Conversation
- .editorconfig (root): UTF-8+LF, trim+final-newline, 4-space; markdown no-trim; ps1/bat CRLF; yaml/json 2-space; Makefile/go tab. Existing files not reformatted (anti-scope). - .inputrc: case-insensitive completion, show-all-if-ambiguous/unmodified, colored-stats, arrow-key history-search; includes /etc/inputrc. - setup-linux.sh deploys .inputrc -> $HOME/.inputrc via deploy_file. - tests/inputrc.bats (4): files exist + content + deploy wiring. - README Features + spec tasks.md updated. Closes #133.
The integration container (setup run from ~/dotfiles-repo != ~/.dotfiles) takes setup-linux.sh's copy-branch, which curates which repo files land in $DOTFILES_DIR before deploy_file reads them. .inputrc was added to the deploy list but not the copy list -> 'deploy_file: source missing' -> integration build failed. Add both new root dotfiles to the copy-branch (incomplete-migration class: new file, stale curated caller). The integration test itself caught it.
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
Why
Two small, universally-applicable cross-OS polish items (mathiasbynens-style) from the fresh-eyes audit. Spec:
specs/POLISH-004-cross-os-polish-bundle/.What
.editorconfig(repo root,root=true): UTF-8 + LF, trim trailing whitespace + final newline, 4-space default; markdown keeps trailing whitespace;*.{ps1,psm1,psd1,bat,cmd}→ CRLF; yaml/json → 2-space; Makefile/go → tab. Anti-scope: existing files are NOT reformatted; governs new work..inputrc:completion-ignore-case,show-all-if-ambiguous,show-all-if-unmodified,colored-stats, arrow-key prefix history-search;$include /etc/inputrc. Deployed to$HOME/.inputrcbysetup-linux.sh(deploy_file).AC status (#133)
.editorconfigexists (editorconfig-checker CI enforcement is out-of-scope per the issue)..inputrcexists;setup-linux.shdeploys to$HOME/.inputrc.tests/inputrc.batsasserts deployment wiring + content (4 tests).Verification
bats tests/inputrc.bats→ 4/4bash -n setup-linux.shOK; shellcheck clean on the new lineCloses #133.