-
-
Notifications
You must be signed in to change notification settings - Fork 14.7k
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
kimai, nixos/kimai: init at 2.24.0 #353187
Conversation
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.
Please format the completely new files with nixfmt
from nixfmt-rfc-style
for the CI formatting check to be happy.
bbeb186
to
e162e26
Compare
Please rebase on master, x86_64-linux build on ofborg is failing due to missing 2f7b6eb. |
e162e26
to
a728c81
Compare
Rebased against current master. Please take another look. |
Can you resolve the merge conflict? I'll try to give this another review later this week. |
a728c81
to
d8d2ab1
Compare
Rebased and solved conflict. Please have another look. |
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.
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.
Ah, branch-off for 24.11 already happened so this won't make it into 24.11 via the master branch.
Are you fine with this going into stable 25.05 instead? Then please move the changelog entry to the new 25.05 entries file.
Otherwise, we'll need to do a backport of this.
d8d2ab1
to
986a038
Compare
25.05 is OK for me. Changelog is moved to 25.05 entries. |
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!
Kimai is a web-based multi-user time-tracking application. Works great for
everyone: freelancers, companies, organizations - everyone can track their
times, generate reports, create invoices and do so much more.
https://www.kimai.org/
The project is built using
buildComposerProject
helper. The module and the test are written following (primarily) Wordpress and Zoneminder.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.