Skip to content

Proofreading the new Azure deployment article #3968

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

Merged
merged 3 commits into from
Jul 1, 2014
Merged

Proofreading the new Azure deployment article #3968

merged 3 commits into from
Jul 1, 2014

Conversation

weaverryan
Copy link
Member

Hi guys!

Q A
Doc fix? no
New docs? no
Applies to 2.3
Fixed tickets -

This is a proofread of #3963 - just with minor (non-technical) changes for readability, etc. I made a quite a lot of changes, so please let me know where I made some mistakes - which I almost definitely have :). Ping also @hhamon!

Thanks!


.. note::

Choosing a more recent PHP version can greatly improve runtime performance.
PHP 5.5 ships with a new built-in PHP accelerator called ZendOptimizer+ that
replaces the APC tool. On an Azure Website, ZendOptimizer+ is already
enabled and there is no need to install and setup any APC server.
replaces APC. On an Azure Website, ZendOptimizer+ is already enabled
Copy link
Member

Choose a reason for hiding this comment

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

the extension is named OPCache since its inclusion in PHP 5.5, not ZendOptimizer+.

Copy link
Contributor

Choose a reason for hiding this comment

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

Good catch!

Many minor changes to make things shorter and just a little more lively :)
@weaverryan
Copy link
Member Author

Thanks for all the comments guys - it helps immensely . Cheers!

@weaverryan weaverryan merged commit 5115183 into 2.3 Jul 1, 2014
weaverryan added a commit that referenced this pull request Jul 1, 2014
This PR was merged into the 2.3 branch.

Discussion
----------

Proofreading the new Azure deployment article

Hi guys!

| Q             | A
| ------------- | ---
| Doc fix?      | no
| New docs?     | no
| Applies to    | 2.3
| Fixed tickets | -

This is a proofread of #3963 - just with minor (non-technical) changes for readability, etc. I made a quite a lot of changes, so please let me know where I made some mistakes - which I almost definitely have :). Ping also @hhamon!

Thanks!

Commits
-------

5115183 Lots of fixes thanks to @xabbuh and @stof!
efb8089 Fixing typo
240ef9d Proofreading the new Azure deployment article
@weaverryan weaverryan deleted the azure branch July 1, 2014 19:56
@hhamon
Copy link
Contributor

hhamon commented Jul 1, 2014

Excellent, thanks!

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.

4 participants