https://bugzilla.redhat.com/show_bug.cgi?id=2238567
Bug ID: 2238567 Summary: Review Request: rust-timeago - Given a Duration, lossily format it like in 'N days ago' Product: Fedora Version: rawhide Status: NEW Component: Package Review Assignee: nobody@fedoraproject.org Reporter: decathorpe@gmail.com QA Contact: extras-qa@fedoraproject.org CC: package-review@lists.fedoraproject.org Target Milestone: --- Classification: Fedora
Spec URL: https://decathorpe.fedorapeople.org/rust-timeago.spec SRPM URL: https://decathorpe.fedorapeople.org/rust-timeago-0.4.1-1.fc38.src.rpm
Description: Given a Duration, lossily format it like in 'N days ago'. Parsing it back to Duration is not supported yet.
Fedora Account System Username: decathorpe
https://bugzilla.redhat.com/show_bug.cgi?id=2238567
--- Comment #1 from Fabio Valentini decathorpe@gmail.com --- This package built on koji: https://koji.fedoraproject.org/koji/taskinfo?taskID=106084680
https://bugzilla.redhat.com/show_bug.cgi?id=2238567
Fabio Valentini decathorpe@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Blocks| |2238264
Referenced Bugs:
https://bugzilla.redhat.com/show_bug.cgi?id=2238264 [Bug 2238264] Migrate to active fork `eza`
https://bugzilla.redhat.com/show_bug.cgi?id=2238567
Michel Lind michel@michel-slm.name changed:
What |Removed |Added ---------------------------------------------------------------------------- Flags| |fedora-review? Assignee|nobody@fedoraproject.org |michel@michel-slm.name Status|NEW |ASSIGNED Doc Type|--- |If docs needed, set a value CC| |michel@michel-slm.name
https://bugzilla.redhat.com/show_bug.cgi?id=2238567
--- Comment #2 from Michel Lind michel@michel-slm.name --- The Apache license text is not present - I'm not sure how serious an issue this is, but otherwise this is good to go
https://bugzilla.redhat.com/show_bug.cgi?id=2238567
--- Comment #3 from Fabio Valentini decathorpe@gmail.com --- Good catch, I somehow missed this when I made the package. This seems to be an oversight by upstream, I'll file an issue.
https://bugzilla.redhat.com/show_bug.cgi?id=2238567
--- Comment #4 from Fabio Valentini decathorpe@gmail.com --- https://github.com/vi/timeago/issues/30
https://bugzilla.redhat.com/show_bug.cgi?id=2238567
--- Comment #5 from Fabio Valentini decathorpe@gmail.com --- Updated to version 0.4.2. This version includes my PR to fix the missing license file.
Spec URL: https://decathorpe.fedorapeople.org/rust-timeago.spec SRPM URL: https://decathorpe.fedorapeople.org/rust-timeago-0.4.2-1.fc38.src.rpm
https://bugzilla.redhat.com/show_bug.cgi?id=2238567
Michel Lind michel@michel-slm.name changed:
What |Removed |Added ---------------------------------------------------------------------------- Flags|fedora-review? |fedora-review+ Status|ASSIGNED |POST
--- Comment #6 from Michel Lind michel@michel-slm.name --- Everything else checks out; APPROVED
https://bugzilla.redhat.com/show_bug.cgi?id=2238567
--- Comment #7 from Fabio Valentini decathorpe@gmail.com --- Thanks for the review!
https://bugzilla.redhat.com/show_bug.cgi?id=2238567
--- Comment #8 from Fedora Admin user for bugzilla script actions fedora-admin-xmlrpc@fedoraproject.org --- The Pagure repository was created at https://src.fedoraproject.org/rpms/rust-timeago
https://bugzilla.redhat.com/show_bug.cgi?id=2238567
Fabio Valentini decathorpe@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Resolution|--- |RAWHIDE Fixed In Version| |rust-timeago-0.4.2-1.fc40 Status|POST |CLOSED Last Closed| |2023-09-25 17:26:27
--- Comment #9 from Fabio Valentini decathorpe@gmail.com --- Imported and built: https://bodhi.fedoraproject.org/updates/FEDORA-2023-f570a923c2
package-review@lists.fedoraproject.org