Skip to content
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

Save database on wp-env. #20648

merged 1 commit into from Mar 6, 2020


Copy link

torounit commented Mar 5, 2020

closes #20657


wp-env does not save the database now.
Each time run wp-env start , the database is emptied and WordPress is newly installed.
should reset the database only when run wp-env clean .

How has this been tested?

  1. $ ./packages/env/bin/wp-env start
  2. add post.
  3. $./packages/env/bin/wp-env stop
  4. $ ./packages/env/bin/wp-env start and check your WordPress.


  • My code is tested.
  • My code follows the WordPress code style.
  • My code follows the accessibility standards.
  • My code has proper inline documentation.
  • I've included developer documentation if appropriate.
  • I've updated all React Native files affected by any refactorings/renamings in this PR.
Copy link

noahtallen left a comment

Good catch!

I verified that before, nothing would be persisted between runs.
With this PR applied, the database persists (as does the session, which is very nice)



This comment has been minimized.

Copy link

noahtallen commented Mar 5, 2020

I might try a rebase to get the tests passing. (I don't have the right permissions to merge with failing checks :))


This comment has been minimized.

Copy link

talldan commented Mar 6, 2020

I've restarted the failing jobs. 🤞

@talldan talldan merged commit 842f4b0 into WordPress:master Mar 6, 2020
3 checks passed
3 checks passed
Travis CI - Pull Request Build Passed
@github-actions github-actions bot added this to the Gutenberg 7.7 milestone Mar 6, 2020
@torounit torounit deleted the torounit:add/env-save-mysql-on-stop branch Mar 6, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
None yet
Linked issues

Successfully merging this pull request may close these issues.

4 participants
You can’t perform that action at this time.