-
-
Notifications
You must be signed in to change notification settings - Fork 1.3k
Update base16 URLs to link to community run version #2871
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.
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
base: master
Are you sure you want to change the base?
Conversation
3d42af2
to
7f1b4ec
Compare
@mk12 Only if you find the time... does this look good to you? @JamyGolden This only changes links. But the actual base16 themes in the repo are not touched? |
@sharkdp yes that is correct, themes are not touched in this PR. The "base16" and "base16-256" theme in the bat repo allow the ansi colors, set by base16-shell, to be automatically applied to bat. This change just updates the readme links so if people discover base16 through bat they can use the maintained and community version of it. |
@sharkdp It doesn't make much difference as far as bat is concerned, I think @JamyGolden is basically requesting a vote in favor of their fork as the actively maintained project. From a cursory look, it seems that the chriskempson repo is not really maintained (no commits in 2 years), and he has been a poor collaborator, deleting repos without warning, not responding to contributors, blocking people. In fact I was annoyed recently that I couldn't find a base16 issue I wanted to refer to — now I realize Chris deleted it. So I'm in favor of merging this. (Aside: I always found the whole base16 ecosystem to be way too complicated for my needs. All I care about is having a convention for CLI programs to use the 16 color palette in a way that works with both dark and light terminal themes, i.e. treat 0 and 7 as background/foreground instead of black/white. And then if you need a few more slots, you can use some from the full 256 palette. That's why base16 has a standard variant and 256 variant. Base16-shell is a hack for terminal emulators that don't let you customize the full 256 colors. I use the kitty terminal, which supports customizing all of them, so I use my own setup https://github.com/mk12/base16-kitty.) |
@sharkdp I've resolved the CHANGELOG conflict, are you happy to merge this PR? |
@sharkdp I've resolved the conflicts again, so should be ready to merge if you're happy with it 😅 (failing windows binary build shouldn't be related to this PR since this PR just includes |
Tinted Theming (previously base16-project) is a maintained and community run version of base16: tinted-theming/home#51
A discussion around creating a base16 organization originally took place in 2016 on an issue in the original base16 repo: https://web.archive.org/web/20220603173440/https://github.com/chriskempson/base16/issues/74. The repo was subsequently nuked and most git history as well as PRs and issues were destroyed (which is why I linked the web archive version of the issue).
PR description
This PR updates the following links:
https://github.com/chriskempson/base16
->https://github.com/tinted-theming/home
https://github.com/chriskempson/base16/blob/master/styling.md
->https://github.com/tinted-theming/home/blob/main/styling.md
https://github.com/chriskempson/base16-shell
->https://github.com/tinted-theming/base16-shell
Our base16-shell repo is updated weekly with the latest schemes.