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
feedgnuplot: init at 1.49 #42668
feedgnuplot: init at 1.49 #42668
Conversation
@GrahamcOfBorg build feedgnuplot |
Success on x86_64-linux (full log) Attempted: feedgnuplot Partial log (click to expand)
|
Success on x86_64-darwin (full log) Attempted: feedgnuplot Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: feedgnuplot Partial log (click to expand)
|
Looks like this doesn't really run any tests because of a gnuplot version mismatch:
So I suggest you either try get the tests to run, or disable them with a comment explaining why. |
I tried to run the plot tests but they fail because they check if rendered outputs are precisely equal to those expected using gnuplot 4.6.4. gnuplot 5's rendering slightly changed. That's why the tests are skipped.
Looking at the homepage and manual, they do not mention about gnuplot's version requirement. Actually it runs fine with gnuplot 5 in my environment. |
In this situation I would suggest to disable tests with a comment explaning why. This saves other people a little time to understand what's going on.
|
Got it. Disabled tests. |
Thank you. |
Success on x86_64-linux (full log) Attempted: feedgnuplot Partial log (click to expand)
|
Success on x86_64-darwin (full log) Attempted: feedgnuplot Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: feedgnuplot Partial log (click to expand)
|
Motivation for this change
It is a nice tool for doing data-science things in command line.
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)