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
rss-bridge: init at 2020-02-26 #81467
Conversation
@@ -0,0 +1,31 @@ | |||
{ config, lib, pkgs, fetchFromGitHub, stdenv, ... }: |
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.
{ config, lib, pkgs, fetchFromGitHub, stdenv, ... }: | |
{ config, lib, pkgs, fetchFromGitHub, stdenv, ... }: | |
747a0d9
to
45b0d7b
Compare
@infinisil do you mind taking a loot at the module? |
also, how can we test this? |
FWIW, I tested this and it works |
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.
Other than the platform
missing, this looks good to me and is ready to merge!
description = "The RSS feed for websites missing it"; | ||
homepage = "https://github.com/RSS-Bridge/rss-bridge"; | ||
license = licenses.unlicense; | ||
maintainers = with maintainers; [ dawidsowa ]; |
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.
platforms
is missing
Motivation for this change
Add rss-bridge.
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)