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
metabase: service module and test #66399
Conversation
@GrahamcOfBorg test metabase |
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.
A few notes.
Hehe, reviewed by multiple people at the same time |
@GrahamcOfBorg test metabase |
@GrahamcOfBorg test metabase |
b6e1eac
to
6e5f622
Compare
@GrahamcOfBorg test metabase |
@GrahamcOfBorg test metabase |
Commits are a bit messed up, adding yourself as a maintainer should be the first one, and it shouldn't include other parts of the PR |
@infinisil sorry, my last force push did not go thought, here we go. :) |
@GrahamcOfBorg test metabase |
ping @infinisil @aanderse |
Awesome. If I could only ask one more thing it would be to drop the unused |
daac315
to
7ce6b15
Compare
Motivation for this change
Adding service module along with the test for metabase.
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)Notify maintainers
cc @schneefux @thoughtpolice