New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Replaced all occurences of e.message with str(e) #3970

Merged
merged 3 commits into from Sep 27, 2018

Conversation

Projects
None yet
2 participants
@schnuffle

schnuffle commented Sep 27, 2018

Signed-off-by: Schnuffle schnuffle@github.com

Schnuffle added some commits Sep 27, 2018

Schnuffle
Replaced all occurences of e.message with str(e)
Signed-off-by: Schnuffle  <schnuffle@github.com>
Schnuffle
@schnuffle

This comment has been minimized.

schnuffle commented Sep 27, 2018

PEP 352 and
PEP 348 might be relevant

@hawkowl

This comment has been minimized.

Contributor

hawkowl commented Sep 27, 2018

Thanks, looks good. I've corrected a typo in the changelog, I'll merge when CI finishes.

@hawkowl hawkowl self-assigned this Sep 27, 2018

@hawkowl hawkowl merged commit a512e63 into matrix-org:develop Sep 27, 2018

5 checks passed

ci/circleci: sytestpy2merged Your tests passed on CircleCI!
Details
ci/circleci: sytestpy2postgresmerged Your tests passed on CircleCI!
Details
ci/circleci: sytestpy3merged Your tests passed on CircleCI!
Details
ci/circleci: sytestpy3postgresmerged Your tests passed on CircleCI!
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details

@schnuffle schnuffle deleted the schnuffle:develop-py3 branch Oct 2, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment