While this /works/ at runtime, it annoys the crap out of static analysis tools. We know what HTTP status code 200 is
- Group Reviewers
- rMSCAad1d5b45f224: Use 200 instead of requests.codes.ok
- Patch without arc
- git checkout -b D1044 && curl -L https://phabricator.wikimedia.org/D1044?download=true | git apply
Future versions of requests probably won't (but totally could) change this from an int to a string or and object or something else to fit their future development needs. Don't think this is necessary.