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: add support for apple color list / .clr files #87

Merged
merged 12 commits into from
Sep 8, 2024

Conversation

uncenter
Copy link
Member

@uncenter uncenter commented Jul 21, 2024

Brings https://github.com/uncenter/ctp-clr to this repository. I would like to add instructions for how exactly to use these files but it doesn't look like we have any existing instructions for the other palette files / apps.

@uncenter uncenter marked this pull request as ready for review July 21, 2024 01:24
scripts/build_palettes.ts Outdated Show resolved Hide resolved
@uncenter uncenter requested a review from sgoudham September 7, 2024 17:06
Copy link
Contributor

@sgoudham sgoudham left a comment

Choose a reason for hiding this comment

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

While fixing the CI errors locally, I realised that this PR breaks the deno task build:palettes command unless you are on macOS.

I think it would be better, while more effort, to extract out the swift compilation into another step that isn't ran as part of the build:palettes command by default since it currently prevents the command from succeeding.

Might even be as simple as checking if an environment variable COMPILE_APPLE_COLOR_LIST is set to 1 or something.

@uncenter uncenter requested a review from sgoudham September 8, 2024 13:45
.github/workflows/test.yml Outdated Show resolved Hide resolved
@sgoudham sgoudham merged commit 88e2795 into catppuccin:main Sep 8, 2024
1 check passed
@uncenter uncenter deleted the feat/clr branch September 8, 2024 16:34
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