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
codimd: 1.2.0 -> 1.4.0, repackage with yarn2nix #60429
Conversation
Awesome! I didn't have a detailed look at this (yet), but that looks much simpler than our previous hackery :) |
Seems like this does in fact not work with hydra, I'll rework this accordingly 😕 |
Now using the most recent version of yarn2nix with hydra support and also rebased on master. @GrahamcOfBorg test codimd |
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.
As the change drastically simplifies the complexity of the codimd
-package itself, I'm all in favor of it 👍
However I guess that we should get feedback from some more folks that were involved in NPM packaging in the past regarding the (re)introduction of yarn2ni x:)
@GrahamcOfBorg test codimd |
(updated)
In this PR
yarn2nix
is used as a replacement fornode2nix
to package codimd, which significantly reduces the complexity of the package definition, thus reducing the overhead of package updates.CodiMD is now packaged with nodejs-10_x.
To avoid the implication of being the only yarn2nix, I added moretea/yarn2nix as
yarn2nix-moretea
. The source is simply copied to nixpkgs so that hydra is able to build packages using mkYarnPackage.