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
go-junit-report: init at 2018-06-14 (385fac0ce9a) #48318
Conversation
yea, I guessed that won't work for me but I had to try.. 😅 |
00c78d4
to
f8ed805
Compare
squashed and rebased on master. used this before nix and now over the weekend on a nix from jenkins witht his in its |
# TODO: goDeps = ./deps.nix; | ||
|
||
meta = with stdenv.lib; { | ||
description = "Converts go test output to an xml report, suitable for applications that expect junit xml reports (e.g. Jenkins)."; |
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.
Can you remove the period at the end of the description please?
(as per nixpkgs manual)
@GrahamcOfBorg build go-junit-report |
Success on aarch64-linux (full log) Attempted: go-junit-report Partial log (click to expand)
|
Will update the description. While I'm at it: what about the version/date in the commit title? The package doesn't have a clear version published so I chose the date of the latest commit. |
also removed the |
Success on x86_64-linux (full log) Attempted: go-junit-report Partial log (click to expand)
|
Your commit message and package naming is good as it is. |
f8ed805
to
219c77b
Compare
perfect - just pushed the cleaned up commit and rebased it on master. thanks for the review!! |
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.
nox-review ok
lrwxrwxrwx 1 xxxxx users 83 27 oct. 15:52 result-bin -> /nix/store/l91v09dgdlydq87avvkmvlip62g33281-go-junit-report-unstable-2018-06-14-bin
Thanks for the contribution
Motivation for this change
tl;dr: pipe
go test
results into this and get enkins reports.Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)nix path-info -S
before and after)