|
From: | Rodger Fox |
Subject: | Re: [PATCH] gnu: Add lmms |
Date: | Thu, 23 Feb 2017 10:13:54 -0800 |
User-agent: | Roundcube Webmail/1.0.6 |
It's a good idea to run "./pre-inst-env guix lint PACKAGENAME" which will pick up long lines for example. I have a ruler/colored line at character 83 in vim, but I'm not yet ready with the setup to write correct code again in vim. There's a similar thing for emacs. I would replace the third or second "It" with @code{LMMS} or simply LMMS.
Thanks for pointing that out. I missed that pre-inst-env tool in the manual until you just mentioned it here. That helps a lot. My corrected patch is attached. It passes the linter this time. -Rodger Fox
0001-gnu-Add-lmms.patch
Description: Text Data
[Prev in Thread] | Current Thread | [Next in Thread] |