https://bugzilla.redhat.com/show_bug.cgi?id=2267483
Bug ID: 2267483 Summary: Review Request: francis - Francis uses the well-known pomodoro technique to help you get more productive Product: Fedora Version: rawhide Status: NEW Component: Package Review Assignee: nobody@fedoraproject.org Reporter: farchord@gmail.com QA Contact: extras-qa@fedoraproject.org CC: package-review@lists.fedoraproject.org Target Milestone: --- Classification: Fedora
Spec URL: https://farchord.fedorapeople.org/reviews/francis/francis.spec SRPM URL: https://farchord.fedorapeople.org/reviews/francis/francis-1.1.0-1.fc40.src.r...
Description: Francis uses the well-known pomodoro technique to help you get more productive.
https://bugzilla.redhat.com/show_bug.cgi?id=2267483
Steve Cossette farchord@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Comment|0 |updated
--- Comment #0 has been edited ---
Spec URL: https://farchord.fedorapeople.org/reviews/francis/francis.spec SRPM URL: https://farchord.fedorapeople.org/reviews/francis/francis-1.1.0-1.fc40.src.r...
Description: Francis uses the well-known pomodoro technique to help you get more productive.
Fedora Account System Username: farchord
https://bugzilla.redhat.com/show_bug.cgi?id=2267483
Neal Gompa ngompa13@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Assignee|nobody@fedoraproject.org |ngompa13@gmail.com Flags| |fedora-review? Status|NEW |ASSIGNED CC| |ngompa13@gmail.com Doc Type|--- |If docs needed, set a value
--- Comment #1 from Neal Gompa ngompa13@gmail.com --- Taking this review.
https://bugzilla.redhat.com/show_bug.cgi?id=2267483
--- Comment #2 from Neal Gompa ngompa13@gmail.com --- Initial spec review:
Summary: Francis uses the well-known pomodoro technique to help you get more productive
... What? Also, this is horrible summary. What *is it*?
Use "URL: https://apps.kde.org/francis/"
Also... write a real description.
https://bugzilla.redhat.com/show_bug.cgi?id=2267483
--- Comment #3 from Steve Cossette farchord@gmail.com --- Spec URL: https://farchord.fedorapeople.org/reviews/francis/francis.spec SRPM URL: https://farchord.fedorapeople.org/reviews/francis/francis-1.1.0-1.fc40.src.r...
Updated Source/summary/description
https://bugzilla.redhat.com/show_bug.cgi?id=2267483
--- Comment #4 from Neal Gompa ngompa13@gmail.com ---
Summary: Francis is a time tracking app for Plasma 6.
Drop the name of the app and the period. Also, is it *only* for Plasma 6?
%description Francis is a time tracking app using the well-known \ pomodoro technique to help you get more productive.
This is already a multi-line macro, you don't need ``.
Url:
Please use "URL:"
https://bugzilla.redhat.com/show_bug.cgi?id=2267483
--- Comment #5 from Steve Cossette farchord@gmail.com --- Spec URL: https://farchord.fedorapeople.org/reviews/francis/francis.spec SRPM URL: https://farchord.fedorapeople.org/reviews/francis/francis-1.1.0-1.fc40.src.r...
More description/url/summary changes
https://bugzilla.redhat.com/show_bug.cgi?id=2267483
Neal Gompa ngompa13@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Summary|Review Request: francis - |Review Request: francis - |Francis is a time tracking |Time tracking app for KDE |app for Plasma 6. |Plasma
https://bugzilla.redhat.com/show_bug.cgi?id=2267483
--- Comment #6 from Neal Gompa ngompa13@gmail.com --- The package is missing "Requires: hicolor-icon-theme".
https://bugzilla.redhat.com/show_bug.cgi?id=2267483
--- Comment #7 from Steve Cossette farchord@gmail.com --- Spec URL: https://farchord.fedorapeople.org/reviews/francis/francis.spec SRPM URL: https://farchord.fedorapeople.org/reviews/francis/francis-1.1.0-1.fc40.src.r...
Updated requires
https://bugzilla.redhat.com/show_bug.cgi?id=2267483
Fedora Review Service fedora-review-bot@fedoraproject.org changed:
What |Removed |Added ---------------------------------------------------------------------------- URL| |https://apps.kde.org/franci | |s/
--- Comment #8 from Fedora Review Service fedora-review-bot@fedoraproject.org --- Copr build: https://copr.fedorainfracloud.org/coprs/build/7099419 (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=2267483
--- Comment #9 from Fedora Review Service fedora-review-bot@fedoraproject.org --- Copr build: https://copr.fedorainfracloud.org/coprs/build/7099422 (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=2267483
--- Comment #10 from Fedora Review Service fedora-review-bot@fedoraproject.org --- Copr build: https://copr.fedorainfracloud.org/coprs/build/7099418 (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=2267483
--- Comment #11 from Fedora Review Service fedora-review-bot@fedoraproject.org --- Copr build: https://copr.fedorainfracloud.org/coprs/build/7099416 (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=2267483
Neal Gompa ngompa13@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flags|fedora-review? |fedora-review+ Status|ASSIGNED |POST
--- Comment #12 from Neal Gompa ngompa13@gmail.com --- Review notes:
* Package follows Fedora Packaging Guidelines * Package builds and installs * Package licensing is correctly handled * No serious issues from rpmlint
PACKAGE APPROVED.
https://bugzilla.redhat.com/show_bug.cgi?id=2267483
--- Comment #13 from Steve Cossette farchord@gmail.com --- Thank you for the review, Neal
https://bugzilla.redhat.com/show_bug.cgi?id=2267483
--- Comment #14 from Fedora Admin user for bugzilla script actions fedora-admin-xmlrpc@fedoraproject.org --- The Pagure repository was created at https://src.fedoraproject.org/rpms/francis
https://bugzilla.redhat.com/show_bug.cgi?id=2267483
Steve Cossette farchord@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|POST |CLOSED Resolution|--- |RAWHIDE Last Closed| |2024-03-03 20:12:28
package-review@lists.fedoraproject.org