Summary

Remove warnings in python 3

Review Request #10207 — Created Oct. 9, 2018 and discarded Aug. 18, 2020, 11:37 a.m.

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	Oct. 9, 2018, 11:16 a.m.
Shouldn't this be p.communicate() ? See Python 2.7 docs and Python 3.7 docs.	brennie	Oct. 9, 2018, 11:16 a.m.

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.

Review Board 7.0 alpha 0 (dev)