Skip to content

helix: add support for path and string themes#6814

Merged
khaneliman merged 1 commit intonix-community:masterfrom
0xda157:helix-theme
Apr 13, 2025
Merged

helix: add support for path and string themes#6814
khaneliman merged 1 commit intonix-community:masterfrom
0xda157:helix-theme

Conversation

@0xda157
Copy link
Copy Markdown
Contributor

@0xda157 0xda157 commented Apr 13, 2025

Description

The waybar module has similar functionality. Adding this because stylix team would like to be able to use a string/path for the theme as disused in nix-community/stylix#1125 (comment)

Checklist

  • Change is backwards compatible.

  • Code formatted with ./format.

  • Code tested through nix-shell --pure tests -A run.all
    or nix build --reference-lock-file flake.lock ./tests#test-all using Flakes.

  • Test cases updated/added. See example.

  • Commit messages are formatted like

    {component}: {description}
    
    {long description}
    

    See CONTRIBUTING for more information and recent commit messages for examples.

  • If this PR adds a new module

    • Added myself as module maintainer. See example.

Maintainer CC

@Philipp-M

@khaneliman khaneliman merged commit 4f898f3 into nix-community:master Apr 13, 2025
3 checks passed
@0xda157 0xda157 deleted the helix-theme branch April 13, 2025 19:19
@0xda157
Copy link
Copy Markdown
Contributor Author

0xda157 commented Apr 13, 2025

hmm seems to be broken for multiple reasons, currently working on fixing it.

@0xda157 0xda157 mentioned this pull request Apr 13, 2025
6 tasks
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