Skip to content

Conversation

@hugovk
Copy link
Member

@hugovk hugovk commented Dec 4, 2014

Review edits for #1032

  • English changes.
  • Renamed filename to 2.7.0 because we may have a 2.7.1.
  • Similarly, refer to 2.7.0 in the document rather than 2.7, and "Pillow 2.7.0" just so there's no confusion with Python 2.7.x.

@hugovk hugovk mentioned this pull request Dec 4, 2014
@homm
Copy link
Member

homm commented Dec 4, 2014

Number also should be changed in docs/releasenotes/index.rst.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

extra )

@hugovk
Copy link
Member Author

hugovk commented Dec 4, 2014

@homm Thanks, updated.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"versions" is more correct, I think.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Both are fine. The performance measurements were measured against only the previous version, not all previous versions.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok, agree.

hugovk added a commit that referenced this pull request Dec 27, 2014
@hugovk hugovk merged commit 7e853c8 into python-pillow:master Dec 27, 2014
@hugovk hugovk deleted the release-notes branch December 27, 2014 09:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants