Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=895077 Fabian Affolter <mail@xxxxxxxxxxxxxxxxxx> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |mail@xxxxxxxxxxxxxxxxxx --- Comment #4 from Fabian Affolter <mail@xxxxxxxxxxxxxxxxxx> --- Just some quick comments: - %dos are missing (e.g. LICENSE-MIT, README.md) - The %description and the %summary needs some tweaking. At the moment both are not very useful. - There is no *.egg-info file in the upstream tarball. I would suggest to leave it in place. You never know what will happen in the future. - There is an folder with examples. Perhaps it's worth to pack those files. The 'review' flag should be set by the reviewer and not the reporter. -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=nuhReVk496&a=cc_unsubscribe _______________________________________________ package-review mailing list package-review@xxxxxxxxxxxxxxxxxxxxxxx https://admin.fedoraproject.org/mailman/listinfo/package-review