Skip to content
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

Added catppuccin themes for bat #3099

Open
wants to merge 7 commits into
base: master
Choose a base branch
from
Open

Conversation

aadirave
Copy link

@aadirave aadirave commented Oct 7, 2024

Added catppuccin themes for bat as a default option in /assets/themes/bat. Done by adding the bat repo from the Catppuccin devs as a submodule. Resolves an issue opened on Sep 4.

@keith-hall
Copy link
Collaborator

Hi, we are considering adding Catppuccin themes to bat. However, ideally not to assets/themes/bat but to assets/themes/Catppuccin, If you can change this and fix the conflicts (including moving the changelog entry to the "unreleased" section) then we would be more likely to merge this PR. Tip: when making changes to a fork, it is always best to create a new branch for it rather than doing it on the default branch.

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