https://bugzilla.redhat.com/show_bug.cgi?id=2291346
Cristian Le <[email protected]> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED CC| |[email protected] URL|https://github.com/Activity |https://github.com/Activity |Watch/%{srcname} |Watch/aw-client Doc Type|--- |If docs needed, set a value Flags| |fedora-review? --- Comment #2 from Cristian Le <[email protected]> --- Some remarks: Are the sed commands necessary? ``` sed -r -i '/^[[:blank:]]*pytest-cov\b/d' pyproject.toml sed -r -i '/^[[:blank:]]*pylint\b/d' pyproject.toml sed -r -i '/^[[:blank:]]*black\b/d' pyproject.toml ``` These seem to be used only in dev dependency, but you manually add `pytest` to it. Otherwise types-* should also be excluded, along with everything else frankly. I'm not a fan of `%{srcname}`, but only `%{_bindir}/%{srcname}` in particular should be changed because there is generally no connection between those. Can you explain the choice for using the git commit and/or link to some discussion upstream on it? --- Other than that LGTM. Would have preferred more tests to be run since the available tests are quite barren, even if they would need to be run outside of package build, but the `input()` would be a bit tricky to deal with. -- 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=2291346 Report this comment as SPAM: https://bugzilla.redhat.com/enter_bug.cgi?product=Bugzilla&format=report-spam&short_desc=Report%20of%20Bug%202291346%23c2 -- _______________________________________________ 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
