-
-
Notifications
You must be signed in to change notification settings - Fork 16.6k
espanso: adding packages from espanso/hub as plugins #403156
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
base: master
Are you sure you want to change the base?
Conversation
This commit requires NixOS/nixpkgs#403156 to be merged into nixpkgs.
This commit requires NixOS/nixpkgs#403156 to be merged into nixpkgs.
This commit requires NixOS/nixpkgs#403156 to be merged into nixpkgs.
This commit requires NixOS/nixpkgs#403156 to be merged into nixpkgs.
Wow, awesome effort here! Please review the PR naming conventions; I think you'll be expected to squash this into two commits: one for adding yourself to the list of maintainers, and a second that will be something along the lines of |
Thank you for the feedback. I have structured the commits as follows:
To streamline the process of committing each plugin, I add the If there are any further suggestions or adjustments needed, please let me know. |
In the recent force-push, I took the time to manually go through all the plugins and add |
bb43a34
to
cf904fd
Compare
Changed my email address and GPG key, so updated the maintainer list accordingly. Rebased the branch onto the latest master. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for your work on this. I haven't been part of init-ing a new plugin system, so would be great to have other maintainers chime in, but it looks good to me.
I’ve reached out on the nixpkgs review request matrix channel to ask someone with experience with plugins in nixpkgs to review this PR. Hopefully, we’ll get more insights soon. |
This commit introduces the `mkEspansoPlugin` function along with an update script that allows users to seamlessly add and update Espanso packages from the [espanso/hub](https://github.com/espanso/hub) repository. The new functionality enables the integration of these packages as plugins under the configuration key `espansoPlugins.<name>`.
Updated the update script to allow specifying the VCS tool for the Fixed an issue with Also ran the update script to update/add packages. |
|
@drupol Sry, to bother you, but would it be possible to merge this PR? Then I could proceed with the home-manager implementation. |
Add Espanso packages from github:espanso/hub as plugins under
espansoPlugins.<name>
. Accordingly, we also introduce aplugins
option to the Espanso Home-Manager service to allow users to easily add these plugins.meta.license
) must be added manually, so I reviewed each package and added the appropriate license metadata accordingly.packages/<package-name>/<most-recent-version>/
to the output directory.Things done
nix.conf
? (See Nix manual)sandbox = relaxed
sandbox = true
nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD"
. Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/
)Add a 👍 reaction to pull requests you find important.