Opened 6 years ago
Closed 6 years ago
#6402 defect closed fixed (fixed)
twisted.words setup metadata is not up to date anymore
| Reported by: | Thijs Triemstra | Owned by: | nothung |
|---|---|---|---|
| Priority: | normal | Milestone: | |
| Component: | words | Keywords: | documentation easy gsoc |
| Cc: | Thijs Triemstra, ralphm | Branch: |
branches/words-setup-metadata-6402
branch-diff, diff-cov, branch-cov, buildbot |
| Author: | rwall |
Description
[source:trunk/twisted/words/topfiles/setup.py]'s long_description and classifiers talk about MSN, OSCAR (AIM & ICQ), TOC (AOL), and X-Ish. All gone by now.
Attachments (2)
Change History (15)
comment:1 Changed 6 years ago by
| Cc: | ralphm added |
|---|
comment:2 Changed 6 years ago by
comment:3 Changed 6 years ago by
| Owner: | set to nothung |
|---|---|
| Status: | new → assigned |
Changed 6 years ago by
| Attachment: | twisted-words-topfiles-setup.patch added |
|---|
patch to Ticket #6402, updates long_description in twisted/words/topfiles/setup.py
comment:4 Changed 6 years ago by
| Keywords: | review added |
|---|
The above patch removes mentions of "MSN" and "TOC (AOL)" from long_description. Kept mentions of "OSCAR (AIM & ICQ)" and "X-ish" following ralphm's suggestion.
Please review this patch for the Outreach Program for Woman (OPW). Thanks!
comment:5 Changed 6 years ago by
| Owner: | nothung deleted |
|---|---|
| Status: | assigned → new |
comment:6 Changed 6 years ago by
| Keywords: | gsoc added; review removed |
|---|---|
| Owner: | set to nothung |
This looks good, but the description probably wants to be reflowed.
Changed 6 years ago by
| Attachment: | twisted-words-topfiles-setup-v2.patch added |
|---|
(revised) patch to Ticket #6402, updates long_description in twisted/words/topfiles/setup.py , (new to this revision) text is reflowed to meet Twisted coding standards
comment:7 Changed 6 years ago by
| Keywords: | review added |
|---|
comment:8 Changed 6 years ago by
| Owner: | nothung deleted |
|---|
comment:10 Changed 6 years ago by
| Author: | → rwall |
|---|---|
| Branch: | → branches/words-setup-metadata-6402 |
(In [38194]) Branching to 'words-setup-metadata-6402'
comment:11 Changed 6 years ago by
comment:12 Changed 6 years ago by
| Keywords: | review removed |
|---|---|
| Owner: | changed from Richard Wall to nothung |
| Status: | assigned → new |
Code Review:
Thanks nothung for rewrapping and submitting a second patch.
I've added it to a branch and fixed a couple of things:
- log:branches/words-setup-metadata-6402
- There was some trailing whitespace in the patch which I have removed. Try setting up your editor to remove trailing whitespace.
See https://twistedmatrix.com/documents/current/core/development/policy/coding-standard.html#auto5
- I added an empty .misc news file - always include a news file in your patches. Although empty files can be difficult with svn diff.
See https://twistedmatrix.com/trac/wiki/ReviewProcess#Newsfiles
Otherwise this looks fine and I'll merge it to trunk.
-RichardW.
comment:13 Changed 6 years ago by
| Resolution: | → fixed |
|---|---|
| Status: | new → closed |
OSCAR support is still there though, and the XMPP support in Words is built on top of xish.