Ticket #1522 task closed fixed

Opened 7 years ago

Last modified 4 years ago

Remove woven from next release of twisted.web

Reported by: itamarst Owned by:
Priority: highest Milestone: Twisted-9.0
Component: web Keywords:
Cc: thijs Branch: branches/remove-woven-1522-2
(diff, github, buildbot, log)
Author: exarkun Launchpad Bug:

Description

Enough is enough.

Change History

1

  Changed 7 years ago by itamarst

  • owner changed from glyph to foom
  • component changed from core to web

2

follow-up: ↓ 4   Changed 7 years ago by exarkun

docs shouldn't be removed until the code is

3

  Changed 7 years ago by jknight

  • owner changed from foom to jknight

4

in reply to: ↑ 2   Changed 5 years ago by collab

  • keywords documentation added
  • cc collab added
  • type changed from enhancement to task

Replying to exarkun:

docs shouldn't be removed until the code is

Where can that code be found?

5

follow-up: ↓ 6   Changed 5 years ago by exarkun

twisted.web.woven :)

therve is working on a branch that removes the use of it from twisted.web.static. Not sure if there are any other uses in Twisted.

6

in reply to: ↑ 5   Changed 5 years ago by collab

  • status changed from new to assigned
  • owner changed from jknight to collab

Replying to exarkun:

twisted.web.woven :) therve is working on a branch that removes the use of it from twisted.web.static. Not sure if there are any other uses in Twisted.

Allright. Also saw pylint mentioning a cyclic import in the woven package.. good to hear its going ;) I could make a patch that scrapes out the documentation for that branch, which one is that?

7

  Changed 5 years ago by thijs

  • owner changed from collab to thijs
  • cc thijs added; collab removed
  • status changed from assigned to new

8

  Changed 5 years ago by thijs

  • owner changed from thijs to therve

assigning it to therve cause he's working on the woven cleanup.

9

  Changed 5 years ago by exarkun

  • keywords documentation removed
  • priority changed from normal to highest
  • summary changed from Remove woven documentation from next release of twisted.web to Remove woven from next release of twisted.web
  • milestone set to Twisted-8.2+1

Also to remove:

  • twisted/web/widgets.py
  • the woven support in twisted/web/tap.py
  • twisted/web/woven/ and associated tests

10

  Changed 4 years ago by exarkun

  • owner changed from therve to exarkun
  • status changed from new to assigned

11

  Changed 4 years ago by exarkun

  • branch set to branches/remove-woven-1522
  • branch_author set to exarkun

(In [25874]) Branching to 'remove-woven-1522'

12

  Changed 4 years ago by exarkun

  • owner exarkun deleted
  • keywords review added
  • status changed from assigned to new

Splat.

13

  Changed 4 years ago by therve

  • keywords review removed
  • owner set to exarkun

Awesome, please merge.

14

  Changed 4 years ago by exarkun

  • branch changed from branches/remove-woven-1522 to branches/remove-woven-1522-2

(In [25918]) Branching to 'remove-woven-1522-2'

15

  Changed 4 years ago by exarkun

  • status changed from new to closed
  • resolution set to fixed

(In [25924]) Merge remove-woven-1522-2

Author: exarkun Reviewer: therve Fixes: #1522

Remove twisted.web.woven and all remaining code which used anything from that package as well as the documentation for Woven.

16

  Changed 2 years ago by <automation>

  • owner exarkun deleted
Note: See TracTickets for help on using tickets.