Summary

Add standard deprecation utilities for RBTools.

Review Request #11517 — Created March 12, 2021 and submitted March 23, 2021, 10:50 p.m.

Information

Owner

david

Repository

RBTools

Branch

master

Bugs

Depends On

Reviewers

Groups

rbtools

People

Description

In other packages, we've started standardizing on utility classes for
emitting deprecation warnings. This changes adds those same utilities to
RBTools, and moves the various warnings that have been recently added in
over to use them.

Testing Done


Ran unit tests.
Triggered a warning and saw it work correctly.

Commits

Summary	ID
Add standard deprecation utilities for RBTools. In other packages, we've started standardizing on utility classes for emitting deprecation warnings. This changes adds those same utilities to RBTools, and moves the various warnings that have been recently added in over to use them. Testing Done: - Ran unit tests. - Triggered a warning and saw it work correctly.	250d33faa68b69bd102f5047c74cec67d75dab85

Issues

Description	From	Last Updated
E501 line too long (80 > 79 characters)	reviewbot	March 12, 2021, 10:10 a.m.
Can you put this in alphabetical order?	chipx86	March 23, 2021, 10:50 p.m.

flake8 failed.

JSHint passed.

flake8

rbtools/clients/__init__.py (Diff revision 1)
The issue has been resolved. Show all issues
```
E501 line too long (80 > 79 characters)
```

Commits:

	Summary	ID
	Add standard deprecation utilities for RBTools. In other packages, we've started standardizing on utility classes for emitting deprecation warnings. This changes adds those same utilities to RBTools, and moves the various warnings that have been recently added in over to use them. Testing Done: - Ran unit tests. - Triggered a warning and saw it work correctly.	062f861a6feee9dfab1ee041096e80e8774dfaaa
	Add standard deprecation utilities for RBTools. In other packages, we've started standardizing on utility classes for emitting deprecation warnings. This changes adds those same utilities to RBTools, and moves the various warnings that have been recently added in over to use them. Testing Done: - Ran unit tests. - Triggered a warning and saw it work correctly.	250d33faa68b69bd102f5047c74cec67d75dab85

Diff:

Revision 2 (+138 -30)

Show changes

	rbtools/deprecation.py
	rbtools/clients/__init__.py
	rbtools/commands/__init__.py

Checks run (2 succeeded)

flake8 passed.

JSHint passed.

Ship it!

rbtools/commands/__init__.py (Diff revision 2)
The issue has been resolved. Show all issues
```
Can you put this in alphabetical order?
```

Status:: Completed
Change Summary:: Pushed to master (62f0989)

rbtools/clients/__init__.py (Diff revision 1)