Skip to content
This repository was archived by the owner on Nov 19, 2024. It is now read-only.

Conversation

dobooth
Copy link
Contributor

@dobooth dobooth commented Jul 25, 2019

Purpose of this pull request

Magento 2.3.3 raises the PHP minimum version and includes PHP 7.3 on the upper end.
This PR updates this version number.
It also removes/updates PHP installation steps and cleaned up some duplicative files.
Lots of markdown cleanup and link fixing.

https://jira.corp.magento.com/browse/MAGEDOC-3966
This pull request (PR) ...

Affected DevDocs pages

https://devdocs.magedevteam.com/661/guides/v2.3/install-gde/system-requirements-tech.html

@magento-cicd2
Copy link
Contributor

magento-cicd2 commented Jul 25, 2019

CLA assistant check
All committers have signed the CLA.

@dobooth dobooth self-assigned this Jul 25, 2019
@dobooth dobooth added xx2.3.3 Magento 2.3.3 changes Technical Updates to the code or processes that alter the technical content of the doc labels Jul 25, 2019
@dobooth dobooth requested a review from jeff-matthews July 29, 2019 19:12
Copy link
Contributor

@hguthrie hguthrie left a comment

Choose a reason for hiding this comment

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

Tread very carefully in Cloud territory.
You must revert many of these changes.

```yaml
build:
flavor: composer
flavor: composer
Copy link
Contributor

Choose a reason for hiding this comment

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

PR #5039 will blow this away

Our YAML files may or may not conform to our linter. It depends on the YAML we refer to in the development repo.
Remove all changes to this file.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done.

1. Find your `php.ini` file using the following command:

php --ini
php --ini
Copy link
Contributor

Choose a reason for hiding this comment

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

Formatting here should be a code block.

3. Change the value of `memory_limit` to at least `1G` for normal use or at least `2G` for debugging.
4. Save your changes to `php.ini` and exit the text editor.
5. Restart your web server:
Use the value of `Loaded Configuration File`.
Copy link
Contributor

Choose a reason for hiding this comment

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

None of these old steps are confirmed accurate. I would drop edits to this old file.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I will revert this file.

Copy link
Contributor

@jeff-matthews jeff-matthews left a comment

Choose a reason for hiding this comment

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

@dobooth

It's very difficult to review PRs this large. Many changes affect formatting only and should be made separately in the future. I stopped commenting on all the formatting instances that require code fences, but that are everywhere.

I'd like to review again after you fix the self-redirect issue that's preventing the php-settings.html file from rendering on staging.

@dobooth
Copy link
Contributor Author

dobooth commented Jul 30, 2019

Removed cloud topics. Fixed code fences. Fixed recursive redirect.

Copy link
Contributor

@hguthrie hguthrie left a comment

Choose a reason for hiding this comment

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

You are hereby released from Cloud lockdown.

@hguthrie hguthrie removed their assignment Jul 30, 2019
@dobooth
Copy link
Contributor Author

dobooth commented Jul 31, 2019

running tests

## PHP (local) {#php}

Install [PHP](https://glossary.magento.com/php) on your local. We recommend PHP 7.1.3+ or 7.2. For information on installing PHP on CentOS and Ubuntu see [PHP](../../../v2.3/install-gde/prereq/php-centos-ubuntu.html). For instructions for another OS, see the [PHP documentation](http://php.net/manual/en/install.php).
Install [PHP](https://glossary.magento.com/php) on your local. We recommend PHP 7.1.3+ or 7.2. For information on installing PHP see [PHP Settings]({{ page.baseurl }}/install-gde/prereq/php-settings.md). For instructions for another OS, see the [PHP documentation](http://php.net/manual/en/install.php).
Copy link
Contributor

Choose a reason for hiding this comment

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

HTML instead of MD

@dobooth
Copy link
Contributor Author

dobooth commented Aug 1, 2019

running tests

@dobooth dobooth merged commit 397f215 into develop Aug 1, 2019
@ghost
Copy link

ghost commented Aug 1, 2019

Hi @dobooth, thank you for your contribution!
Please, complete Contribution Survey, it will take less than a minute.
Your feedback will help us to improve contribution process.

@dobooth dobooth deleted the db_php_upgrade branch August 1, 2019 16:33
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Technical Updates to the code or processes that alter the technical content of the doc xx2.3.3 Magento 2.3.3 changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants