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
libertinus: init at 6.4 #31461
libertinus: init at 6.4 #31461
Conversation
outputHash = "1fjk3vmixl1gmjn0cj5jyc4r933mqm9wrb6kn447nisxmkvn379j"; | ||
|
||
meta = with stdenv.lib; { | ||
description = "Libertinus fonts is a fork of the Linux Libertine and Linux Biolinum fonts that started as an OpenType math companion of the Libertine font family, but grown as a full fork to address some of the bugs in the fonts."; |
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.
The description is too long.
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.
Is there a rule of thumb for how long descriptions should be?
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.
It should fit on one line of a full screen terminal (e.g. in the nox
output), so the 100 character limit is reasonable, with most descriptions being under 80 characters.
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.
Thanks for the pointer!
|
||
stdenv.mkDerivation rec { | ||
name = "libertinus-${rev}"; | ||
rev = "v6.4"; |
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.
The version must start with a digit. Use rev = "v${version}"
below.
33428e6
to
3bbcb68
Compare
@orivej Thanks for the review! I've made the changes you requested. |
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.
The description is needed even if longDescription is provided. I've added one.
Motivation for this change
Another nice OTF math font!
Things done
build-use-sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)