https://bugzilla.redhat.com/show_bug.cgi?id=2051643

Jakub Kadlčík <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[email protected]
           Doc Type|---                         |If docs needed, set a value



--- Comment #1 from Jakub Kadlčík <[email protected]> ---
Hello Alessio,
thank you for the package.

I tried to install, and use it, and it works.


> Version: 2.0.5

There is already version 2.1.0
https://github.com/lite-xl/lite-xl/releases/tag/v2.1.0

That probably eliminates the need for packaging it via %global commit,
I guess? 


> %description
> A lightweight, simple, fast, feature-filled, and extremely
> extensible text editor written in C, and Lua, adapted from lite.

I would probably use the Overview text from README.
https://github.com/lite-xl/lite-xl#overview


> %license %{_docdir}/%{name}/licenses.md

Maybe also %license LICENSE, just to be sure?



Otherwise LGTM.


-- 
You are receiving this mail because:
You are always notified about changes to this product and component
You are on the CC list for the bug.
https://bugzilla.redhat.com/show_bug.cgi?id=2051643
_______________________________________________
package-review mailing list -- [email protected]
To unsubscribe send an email to [email protected]
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/[email protected]
Do not reply to spam, report it: 
https://pagure.io/fedora-infrastructure/new_issue

Reply via email to