Opened 16 years ago

Closed 16 years ago

#1831 task closed fixed (fixed)

Delete twisted.spread.newjelly

Reported by: Jean-Paul Calderone Owned by:
Priority: highest Milestone:
Component: core Keywords:
Cc: warner, Glyph, itamarst, Jean-Paul Calderone Branch:
Author:

Description

We never made the leap to newjelly. I doubt we ever will. Having copy of jelly.py for what seems like a relatively minor refactoring was a bad idea anyway. Fixes for bugs copied into newjelly have been applied to jelly but not newjelly. Let's toss this junk.

Change History (6)

comment:1 Changed 16 years ago by Jean-Paul Calderone

Cc: warner Glyph itamarst Jean-Paul Calderone added
Keywords: review added
Owner: Jean-Paul Calderone deleted
Priority: normalhighest

Ready for review in source:branches/remove-newjelly-1831

comment:2 Changed 16 years ago by radix

Owner: set to Jean-Paul Calderone

+1 (after an agreement from glyph, as well)

The way I read it, you moved some stuff from test_newjelly into test_jelly and removed all the newjelly stuff. right?

comment:3 Changed 16 years ago by radix

Keywords: review removed

comment:4 Changed 16 years ago by Jean-Paul Calderone

Right.

comment:5 Changed 16 years ago by Jean-Paul Calderone

Resolution: fixed
Status: newclosed

(In [17352]) Merge remove-newjelly-1831

Author: exarkun Reviewers: radix, glyph Fixes #1831

Move all of the newjelly tests into the jelly test module and get rid of the newjelly code.

comment:6 Changed 11 years ago by <automation>

Owner: Jean-Paul Calderone deleted
Note: See TracTickets for help on using tickets.