Encode non-string query parameters as strings
Review Request #10685 — Created Sept. 6, 2019 and updated — Latest diff uploaded
There are a few cases where we are passing non-string arguments to
methods that make API calls, which was fine in the past. However, we are
now assuming that all arguments are strings and erroring out if they are
Instead, we now make a best-case attempt to serialize values to strings
and produces a better error message when this process fails.
Confirmed the following hold true on Python versions 2.7, 3.5, 3.6, and
- With this patch applied,
rbt statusprints the list of status, where
previously it would throw an exception due to attempting to stringify
- With this entire patch stack (/r/10695, /r/10696, and /r/10697),
rbt postis able to create review requests with commit history.