This change has been discarded.

Describe the reason it's discarded (optional):

Summary

Remove warnings in python 3

Review Request #10207 — Created Oct. 9, 2018 and discarded 4 years, 6 months ago

Owner

Repository

RBTools

Branch

master

Bugs

Depends On

Groups

People

Description*

Remove warnings in python 3

Testing Done

existing unit tests

Issues

Description	From	Last Updated
Can you flesh your description out to indicate what warnings you are fixing the causes of?	brennie	6 years, 5 months ago
Shouldn't this be p.communicate() ? See Python 2.7 docs and Python 3.7 docs.	brennie	6 years, 5 months ago

flake8 passed.

JSHint passed.

Fix it!

An issue was opened. Show all issues

Can you flesh your description out to indicate what warnings you are fixing the causes of?

rbtools/utils/checks.py (Diff revision 1)

An issue was opened. Show all issues

Shouldn't this be p.communicate() ?
See Python 2.7 docs and Python 3.7 docs.