Skip to content

Bump the Travis CI memory limit to 2048MB#136

Merged
schlessera merged 2 commits intowp-cli:masterfrom
stevegrunwell:fix/raise-travis-memory-limit
Apr 10, 2018
Merged

Bump the Travis CI memory limit to 2048MB#136
schlessera merged 2 commits intowp-cli:masterfrom
stevegrunwell:fix/raise-travis-memory-limit

Conversation

@stevegrunwell
Copy link
Copy Markdown
Contributor

The Behat tests are currently running out of memory in PHP 5.3 environments, as noted by @schlessera in wp-cli/entity-command#161.

This uses the same code as wp-cli/entity-command@f449ea9 to raise the memory_limit within Travis' PHP configuration.

The Behat tests are currently running out of memory in PHP 5.3 environments, as noted by @schlessera in wp-cli/entity-command#161.

This uses the same code as wp-cli/entity-command@f449ea9 to raise the memory_limit within Travis' PHP configuration.
.travis.yml Outdated
echo "xdebug.ini does not exist"
fi
- |
# Raise PHP memory limit to 256MB
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.

This was wrong in my original commit, we actually raise it to 2048MB.

@schlessera schlessera changed the title Bump the Travis CI memory limit to 256MB Bump the Travis CI memory limit to 2048MB Apr 10, 2018
@schlessera
Copy link
Copy Markdown
Member

@stevegrunwell Thanks for the PR. Unfortunately, I had an error in my original commit's comment and you've copied that into this PR as well. As soon as this is redressed, we can merge this.

@schlessera schlessera added the scope:testing Related to testing label Apr 10, 2018
@schlessera schlessera added this to the 1.1.3 milestone Apr 10, 2018
…you copy it from a @schlessera commit.

Updates comment above the line bumping Travis CI's memory_limit to accurately reflect what's happening (2GB, not 256MB).
@stevegrunwell
Copy link
Copy Markdown
Contributor Author

@schlessera D'oh, that's what I get for blindly copying. An updated version of the comment has been pushed.

Thanks for catching that!

@schlessera schlessera merged commit bdafbd9 into wp-cli:master Apr 10, 2018
@stevegrunwell stevegrunwell deleted the fix/raise-travis-memory-limit branch April 10, 2018 13:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

scope:testing Related to testing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants