Skip to content

fix(soe): re-land ls/grep colour as a fix - #132

Merged
catinspace-au merged 2 commits into
mainfrom
fix/soe-ls-colour-reland
Nov 10, 2026
Merged

catinspace-au merged 2 commits into
mainfrom
fix/soe-ls-colour-reland

Conversation

@catinspace-au

Copy link
Copy Markdown
Contributor

#79 landed on main as feat(soe) (7c77f61), so the next release would be a minor. The squash took the single commit's subject, not the PR title.

  • f287134 reverts it. Its subject is revert: plus the quoted original header. semantic-release matches that form to drop both commits from the bump.
  • 888c538 re-applies the same 2 files as fix(soe).

The tree is identical to main.

Merge with REBASE. A squash folds both commits into one and the revert is lost.

#79)"

This reverts commit 7c77f61.

The feat subject would trigger a minor release. The next commit reapplies the same change as a fix. semantic-release drops a reverted commit only when the revert names its exact header and hash.
Colouring is off on any box whose home directory did not come from the distro's own skel. A Fedora-derived .bashrc on Ubuntu leaves LS_COLORS empty and no ls alias, so ls and the prompt render monochrome.

Uses only what each OS ships. GNU ls (Linux, or macOS with brew coreutils) reads LS_COLORS. BSD ls ignores LS_COLORS and reads LSCOLORS. The block detects which ls it has at shell start.

Colours use ANSI indices rather than RGB. They track the terminal theme and need no light/dark handling. Solarized keeps 14 of 16 palette slots identical across its light and dark variants.

The block goes in the interactive shell rc. Aliases are not exported. Ubuntu ships its profile.d hook commented out in /etc/bash.bashrc, so a terminal's non-login shell never reads /etc/profile.d.

System-wide by default, soe_ls_colour_scope=user for a single account.
@catinspace-au
catinspace-au merged commit 403bc44 into main Nov 10, 2026
18 checks passed
@catinspace-au
catinspace-au deleted the fix/soe-ls-colour-reland branch November 10, 2026 07:44
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.

1 participant