( )⚙ D2131 py3: replace file() with open() in test-encoding-align.t

This is an archive of the discontinued Mercurial Phabricator instance.

py3: replace file() with open() in test-encoding-align.t
ClosedPublic

Authored by pulkit on Feb 11 2018, 7:42 AM.

Details

Summary

file() is not present in Python 3.
This patch also makes sure we write bytes in Python 3.

Diff Detail

Repository
rHG Mercurial
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

pulkit created this revision.Feb 11 2018, 7:42 AM
yuja requested changes to this revision.Feb 11 2018, 9:06 AM
yuja added a subscriber: yuja.

Perhaps these open()s need 'b' because we're testing non-ascii strings.

This revision now requires changes to proceed.Feb 11 2018, 9:06 AM
pulkit edited the summary of this revision. (Show Details)Feb 12 2018, 3:48 AM
pulkit updated this revision to Diff 5480.
yuja accepted this revision.Feb 12 2018, 5:45 AM
This revision is now accepted and ready to land.Feb 12 2018, 5:45 AM
This revision was automatically updated to reflect the committed changes.