Remove providing_args from all Signal instances.

Review Request #12017 — Created Jan. 27, 2022 and submitted — Latest diff uploaded

Information

Review Board
master

Reviewers

Signal has historically contained a providing_args= argument, which
listed the names of the arguments expected to be sent to the signal.
This was optional and never used for anything beyond the ability to
introspect a signal (which probably nobody does), and isn't supported on
modern versions of Django.

This change removes providing_args= from all signals.

Ran unit tests.

Diff Revision 1 (Latest)

Commits

First Last Summary ID Author
Remove providing_args from all Signal instances.
`Signal` has historically contained a `providing_args=` argument, which listed the names of the arguments expected to be sent to the signal. This was optional and never used for anything beyond the ability to introspect a signal (which probably nobody does), and isn't supported on modern versions of Django. This change removes `providing_args=` from all signals. Testing Done: Ran unit tests.
be84f4056624c352481f713b8c9f94777ae96ac5 David Trowbridge
reviewboard/signals.py
reviewboard/reviews/signals.py
reviewboard/scmtools/signals.py
reviewboard/site/signals.py
Loading...