镜像站点 · 本页由第三方 GitHub 只读镜像提供,非 GitHub 官方站点,不接受任何登录或凭据输入。前往 github.com
Skip to content

Release note review - #1038

Merged
hugovk merged 3 commits into
python-pillow:masterfrom
hugovk:release-notes
Dec 27, 2014
Merged

hugovk merged 3 commits into
python-pillow:masterfrom
hugovk:release-notes

Conversation

@hugovk

@hugovk hugovk commented Dec 4, 2014

Copy link
Copy Markdown
Member

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

homm commented Dec 4, 2014

Copy link
Copy Markdown
Member

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

Comment thread docs/releasenotes/2.7.0.rst Outdated

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

extra )

@hugovk

hugovk commented Dec 4, 2014

Copy link
Copy Markdown
Member Author

@homm Thanks, updated.

Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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