Fedora Account System
Red Hat Associate
Red Hat Customer
Spec URL: https://nonamedotc.fedorapeople.org/pkgreview/python-packages/2019-2020/engineio/rnd1/python-engineio.spec SRPM URL: https://nonamedotc.fedorapeople.org/pkgreview/python-packages/2019-2020/engineio/rnd1/python-engineio-3.11.1-1.fc31.src.rpm Description: Python implementation of the Engine.IO realtime client and server. Fedora Account System Username: nonamedotc koji scratch build: https://koji.fedoraproject.org/koji/taskinfo?taskID=39893209
The Summary or %description should say what this package is for, at least a little bit. If someone (like me) doesn't know what Engine.IO is, it is hard to figure out anything about the package right now. %description is repeated, defining it once as a macro using %{expand:} would be nice. + package name is OK + license is acceptable for Fedora (MIT) + license is specified correctly + builds and installs OK + fedora-review and rmplint seem happy + R/P/BR look OK Package is APPROVED.
Thanks for the review. I will update the specfile with %{expand:...} macro when I import into dist-git. I will update the description as follows when I import: "Engine.IO is a lightweight transport protocol that enables real-time bidirectional event-based communication between clients (typically, though not always, web browsers) and a server. The official implementations of the client and server components are written in JavaScript. This package provides Python implementations of both, each with standard and asyncio variants." Taken directly from the py-engineio docs.
(fedscm-admin): The Pagure repository was created at https://src.fedoraproject.org/rpms/python-engineio
Built on rawhide.