Ticket #4946 enhancement closed fixed

Opened 2 years ago

Last modified 2 years ago

Deprecate allYourBase and accumulateBases

Reported by: lvh Owned by: lvh
Priority: normal Milestone:
Component: core Keywords: easy
Cc: Branch: branches/deprecate-reflect-stuff-4946
(diff, github, buildbot, log)
Author: lvh Launchpad Bug:

Description

For more details see #4928 -- we've removed the use cases for this except in some code which is going to be deprecated so we should probably deprecate this in favor of fancy inspect module functions.

Attachments

4946.patch Download (2.1 KB) - added by aruse 2 years ago.
Deprecation of allYourBase and accumulateBases

Change History

1

Changed 2 years ago by aruse

  • status changed from new to assigned
  • owner set to aruse

Changed 2 years ago by aruse

Deprecation of allYourBase and accumulateBases

2

Changed 2 years ago by aruse

  • status changed from assigned to new
  • keywords easy, review added; easy removed
  • owner aruse deleted

3

Changed 2 years ago by lvh

  • owner set to lvh
  • status changed from new to assigned
  • keywords easy added; easy, removed

4

Changed 2 years ago by lvh

  • branch set to branches/deprecate-reflect-stuff-4946
  • branch_author set to lvh

(In [31036]) Branching to 'deprecate-reflect-stuff-4946'

5

Changed 2 years ago by lvh

  • keywords review removed

Thanks for your patch! This is great, but blocking on #4928 since until that gets merged Twisted itself still uses allYourBase so it fires its own DeprecationWarning. As soon as that's fixed we should merge branches/deprecate-reflect-stuff-4946 into trunk.

6

Changed 2 years ago by lvh

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

(In [31056]) allYourBase and accumulateBases are now deprecated in favor of inspect.getmro

Author: aruse Reviewer: lvh Fixes: #4946

Note: See TracTickets for help on using tickets.