https://bugzilla.redhat.com/show_bug.cgi?id=2181036 --- Comment #5 from Fabio Valentini <decathorpe@xxxxxxxxx> --- > The README links to the img file, should I flag it as documentation instead of removing it to avoid a broken reference? That's an option. However, there is zero places where users would actually see rendered README files for this package, so removing the image wouldn't make a difference either. > And similarly, it looks like many of the crates include coverage files. Should they all be patched? It's certainly nicer if these files aren't included in our packages, but they aren't harmful. On the other hand, the upstream projects might even be happy about patches (to add "exclude" to Cargo.toml) that make downloads of their crates from crates.io smaller / faster for everyone? :) Either way, neither the image, nor CODEOWNERS, nor coverage data is "harmful", but they are also just bloat. You can remove them, but you don't *have* to. -- 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=2181036 _______________________________________________ package-review mailing list -- package-review@xxxxxxxxxxxxxxxxxxxxxxx To unsubscribe send an email to package-review-leave@xxxxxxxxxxxxxxxxxxxxxxx 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/package-review@xxxxxxxxxxxxxxxxxxxxxxx Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue
