Opened 4 years ago
Closed 4 years ago
#6333 enhancement closed fixed (fixed)
Deprecate DryRunVisitor.
| Reported by: | tom.prince | Owned by: | tom.prince |
|---|---|---|---|
| Priority: | normal | Milestone: | |
| Component: | trial | Keywords: | easy |
| Cc: | jml | Branch: |
branches/deprecate-dryrunvisitor-6333-2
branch-diff, diff-cov, branch-cov, buildbot |
| Author: | tomprince |
Change History (8)
comment:1 Changed 4 years ago by DefaultCC Plugin
- Cc jml added
comment:2 Changed 4 years ago by tomprince
- Branch set to branches/deprecate-dryrunvisitor-6333
comment:3 Changed 4 years ago by tom.prince
- Keywords review added
This should probably land after #3231.
comment:4 Changed 4 years ago by tom.prince
comment:5 Changed 4 years ago by glyph
- Keywords review removed
- Owner set to tom.prince
This looks fine to me, such as it is, but, as with many deprecations in Twisted, I feel like the explanation is not really sufficient. Why are visitors no longer supported? If I had written some code (what code might I have written?) that was using this class, what do I need to do instead of using visitors? If you can think of more useful words to add to the deprecation message explaining why this was done, please do so.
comment:6 Changed 4 years ago by exarkun
This looks fine to me, such as it is, but, as with many deprecations in Twisted, I feel like the explanation is not really sufficient.
I agree.
Why are visitors no longer supported? If I had written some code (what code might I have written?) that was using this class, what do I need to do instead of using visitors? If you can think of more useful words to add to the deprecation message explaining why this was done, please do so.
Visitors were deprecated in #2897, to be replaced by test decorators. I think jml or lifeless should be herded into a closet until they write up some documentation about test decorators.
comment:7 Changed 4 years ago by tomprince
- Branch changed from branches/deprecate-dryrunvisitor-6333 to branches/deprecate-dryrunvisitor-6333-2
(In [37586]) Branching to deprecate-dryrunvisitor-6333-2.
comment:8 Changed 4 years ago by tomprince
- Resolution set to fixed
- Status changed from new to closed
(In [37274]) Branching to deprecate-dryrunvisitor-6333.