https://bugzilla.redhat.com/show_bug.cgi?id=2266273
Bug ID: 2266273 Summary: Review Request: rust-spectral - Fluent test assertions Product: Fedora Version: rawhide Status: NEW Component: Package Review Assignee: nobody@fedoraproject.org Reporter: dmellado@redhat.com QA Contact: extras-qa@fedoraproject.org CC: package-review@lists.fedoraproject.org Target Milestone: --- Classification: Fedora
Spec URL: https://dmellado.fedorapeople.org/rust-spectral/rust-spectral.spec SRPM URL: https://dmellado.fedorapeople.org/rust-spectral/rust-spectral-0.6.0-1.fc39.s...
Description: Fluent test assertions.
Fedora Account System Username: dmellado
https://bugzilla.redhat.com/show_bug.cgi?id=2266273
--- Comment #1 from Daniel Mellado dmellado@redhat.com --- This package built on koji: https://koji.fedoraproject.org/koji/taskinfo?taskID=114126869
https://bugzilla.redhat.com/show_bug.cgi?id=2266273
Fedora Review Service fedora-review-bot@fedoraproject.org changed:
What |Removed |Added ---------------------------------------------------------------------------- URL| |https://crates.io/crates/sp | |ectral
--- Comment #2 from Fedora Review Service fedora-review-bot@fedoraproject.org --- Copr build: https://copr.fedorainfracloud.org/coprs/build/7064471 (succeeded)
Review template: https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora-rev...
Please take a look if any issues were found.
--- This comment was created by the fedora-review-service https://github.com/FrostyX/fedora-review-service
If you want to trigger a new Copr build, add a comment containing new Spec and SRPM URLs or [fedora-review-service-build] string.
https://bugzilla.redhat.com/show_bug.cgi?id=2266273
Fabio Valentini decathorpe@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flags| |fedora-review? Assignee|nobody@fedoraproject.org |decathorpe@gmail.com Status|NEW |ASSIGNED CC| |decathorpe@gmail.com
--- Comment #3 from Fabio Valentini decathorpe@gmail.com --- This package is *ancient* ... are you sure you need this? Or would it be possible to replace it with something that is not dusty and old?
There's two minor issues that would need fixing:
- If you disable tests, please add a short comment in the line preceding "%bcond_with check" that explains the reason. - The downstream patch to bump the "num" dependency from 0.1 to 0.4 needs a short comment too.
https://bugzilla.redhat.com/show_bug.cgi?id=2266273
Daniel Mellado dmellado@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Resolution|--- |WONTFIX Status|ASSIGNED |CLOSED Last Closed| |2024-03-06 16:17:29
--- Comment #4 from Daniel Mellado dmellado@redhat.com --- Closing this as spectral is now considered outdated
package-review@lists.fedoraproject.org