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

wayfire: add module #6066

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

wayfire: add module #6066

wants to merge 1 commit into from

Conversation

0x5a4
Copy link
Contributor

@0x5a4 0x5a4 commented Nov 10, 2024

Description

Adds a Module for the Wayfire Compositor. Also allows managing the wf-shell configuration.

May Close #3826

Checklist

  • Change is backwards compatible.

  • Code formatted with ./format.

  • Code tested through nix-shell --pure tests -A run.all or nix develop --ignore-environment .#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

@0x5a4 0x5a4 force-pushed the init-wayfire branch 3 times, most recently from 5b7d2f2 to c13de2c Compare November 11, 2024 12:16
@0x5a4 0x5a4 marked this pull request as ready for review November 11, 2024 12:16
@0x5a4 0x5a4 force-pushed the init-wayfire branch 3 times, most recently from 6300090 to a7b471a Compare November 11, 2024 12:34
@0x5a4
Copy link
Contributor Author

0x5a4 commented Nov 11, 2024

I dont know why CI is failing, i didnt touch the affected module. The Test also fails on master for me.

@0x5a4 0x5a4 force-pushed the init-wayfire branch 2 times, most recently from bebc4a4 to 27a64e0 Compare November 13, 2024 11:18
@0x5a4
Copy link
Contributor Author

0x5a4 commented Nov 13, 2024

I bundled the wf-shell module because I dont feel like its used outside of wayfire. I'm not sure if thats even possible. Though I understand if you would like me to separate it

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.

Options for wayfire compositor
1 participant