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
epgstation: 1.7.4 -> 1.7.5 #99632
epgstation: 1.7.4 -> 1.7.5 #99632
Conversation
/marvin opt-in |
Hi! I'm an experimental bot. My goal is to guide this PR through its stages, hopefully ending with a merge. You can read up on the usage here. |
b31ff92
to
e2a3a02
Compare
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.
LGTM at first glance.
/marvin needs_merger |
Thanks! JFYI the command for changing statuses is /status needs_merger |
The PR author cannot set the status to If you are not the PR author and you are reading this, please review the usage of this bot. You may be able to help. Please make an honest attempt to resolve all outstanding issues before setting to |
Oh. |
/status needs_merger |
I always mix this up. -.- |
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.
Since there are no conflicts I'd just merge this right now. For follow-up merge conflicts I'd then bundle them |
Okay, thanks! Also thank you for the review @wamserma. |
Motivation for this change
passthru.updateScript
name
fromnode_EPGStation
toEPGStation
.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)