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

feat: ansi 16 theme #22

Merged
merged 7 commits into from
Nov 22, 2024
Merged

feat: ansi 16 theme #22

merged 7 commits into from
Nov 22, 2024

Conversation

meowgorithm
Copy link
Member

@meowgorithm meowgorithm commented Nov 20, 2024

This revision adds a semi-secret theme that uses the 16 ANSI colors for people who like it when CLIs match their colorscheme. It can be enabled by setting SEQUIN_THEME to ansi, matchy, carlos or secret_carlos.

Under the hood, this themes uses the hot, new ANSI constants in Lip Gloss v2 Alpha.

base16 theme screenshot

@coveralls
Copy link

coveralls commented Nov 20, 2024

Pull Request Test Coverage Report for Build 11976924048

Details

  • 80 of 109 (73.39%) changed or added relevant lines in 2 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-2.5%) to 87.883%

Changes Missing Coverage Covered Lines Changed/Added Lines %
main.go 19 23 82.61%
theme.go 61 86 70.93%
Totals Coverage Status
Change from base Build 11976306959: -2.5%
Covered Lines: 660
Relevant Lines: 751

💛 - Coveralls

@aymanbagabas aymanbagabas changed the title feat: base16 theme feat: ansi 16 theme Nov 20, 2024
Copy link
Member

@caarlos0 caarlos0 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

need to move the //nolint:mnd to the new func, other than that lgtm

@caarlos0 caarlos0 merged commit 7c3299f into main Nov 22, 2024
19 checks passed
@caarlos0 caarlos0 deleted the carlos-mode branch November 22, 2024 17:08
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.

4 participants