[Bug 1094040] Review Request: python-taskreport - Automatic reporting tool for Taskwarrior

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



https://bugzilla.redhat.com/show_bug.cgi?id=1094040

Dan Scott <dan@xxxxxxxxxxxxxx> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |dan@xxxxxxxxxxxxxx



--- Comment #1 from Dan Scott <dan@xxxxxxxxxxxxxx> ---
SPEC should use python2_sitelib and python2_sitearch per
https://fedoraproject.org/wiki/Packaging:Python#Macros

I would recommend removing the editorial comments from the description section,
as the opinion doesn't really fit with a package description. I would be
tempted to trim the description all the way down to just "inlinestyler is an
easy way to locally inline CSS into an HTML email message."

But if you really want to keep the longer description, at least trim the
following from:

"""
The general solution is to use an in-lining service, which takes a message with
the CSS placed externally, and rewrites it so that all CSS is applied to the
individual elements. The most widely used of these services - and as far as I
can tell, the one that powers CampaignMonitor - is Premailer. It's a great
service, and the guys behind it put a lot of work into keeping it up to date
with the most recent discoveries in what works and what doesn't.
"""

to:

"""
The general solution is to use an in-lining service, which takes a message with
the CSS placed externally, and rewrites it so that all CSS is applied to the
individual elements. The most widely used of these services is Premailer.
"""

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
_______________________________________________
package-review mailing list
package-review@xxxxxxxxxxxxxxxxxxxxxxx
https://admin.fedoraproject.org/mailman/listinfo/package-review





[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]