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

Fix an issue with the snapshots directory and the command for unzipping the database archive #60

Open
wants to merge 3 commits into
base: develop
Choose a base branch
from

Conversation

kmgalanakis
Copy link

@kmgalanakis kmgalanakis commented Aug 8, 2023

Description of the Change

In this MR I'm doing Windows-related modifications to make Snapshots running in Windows too. I've also updated the installation instructions in the README.md file with some Windows-related steps.

Closes #59

How to test the Change

  • Pull, download, create, push and delete Snapshots from within both a Windows and a Unix machine.

Changelog Entry

Added - Full Windows installation support

Credits

Props @gsarig

Checklist:

  • I agree to follow this project's Code of Conduct.
  • I have updated the documentation accordingly.
  • I have added tests to cover my change.
  • All new and existing tests pass.

@kmgalanakis kmgalanakis marked this pull request as draft August 8, 2023 20:48
@kmgalanakis kmgalanakis force-pushed the fix/snapshots-pull-windows branch 5 times, most recently from 15dabcc to 38afb97 Compare August 9, 2023 00:16
@kmgalanakis kmgalanakis self-assigned this Aug 9, 2023
@kmgalanakis kmgalanakis marked this pull request as ready for review August 9, 2023 07:22
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.

Snapshots WP CLI commands are not executed at all
1 participant