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

Added API To Config Themes #61

Merged
merged 50 commits into from
Nov 4, 2020
Merged

Added API To Config Themes #61

merged 50 commits into from
Nov 4, 2020

Conversation

OmriBarZik
Copy link
Owner

@OmriBarZik OmriBarZik commented Nov 4, 2020

WHAT'S NEW

Added JS API to control WordPress themes Working on #53

Theme

  • Added: API to activate a theme
  • Added: API to delete a theme
  • Added: API to get theme data and info
  • Added: API to install a theme
  • Added: API to check is theme is active
  • Added: API to check is theme is installed
  • Added: API to list installed WordPress themes on current site
  • Added: API to get themes directory path and parent theme directory path

Utils

  • ReturnPromise now return both stderr and stdout of the prosses to the callback

Docker

  • Added RunInContainer API to create a container run a commands and delete the container.

Container

  • Added isHealthy to check if the container is heathy.
  • Added exec to run commands on a running container.
  • Updated API to check the container health
    • transform API to an object
    • options.health.command same as options.healthCommand
    • options.health.interval - the time between running each check.
    • options.health.retries - the number of retries until the docker become un healthy.
    • options.health.startPeriod - time to wait until the testing starts.
    • options.health.timeout - the max time to run health time.

Environment

  • Added SiteSetup to create and initialize a WordPress site

@codecov-io
Copy link

Codecov Report

Merging #61 into main will decrease coverage by 3.53%.
The diff coverage is 88.61%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main      #61      +/-   ##
==========================================
- Coverage   91.92%   88.39%   -3.54%     
==========================================
  Files           9       11       +2     
  Lines         223      293      +70     
==========================================
+ Hits          205      259      +54     
- Misses         18       34      +16     
Impacted Files Coverage Δ
src/workflow/environment.js 0.00% <0.00%> (-46.16%) ⬇️
src/workflow/sites-management.js 0.00% <ø> (ø)
src/docker/container.js 100.00% <100.00%> (ø)
src/docker/docker.js 100.00% <100.00%> (ø)
src/docker/network.js 100.00% <100.00%> (ø)
src/docker/presets/containers.js 100.00% <100.00%> (ø)
src/docker/util.js 100.00% <100.00%> (ø)
src/wp-cli/theme.js 100.00% <100.00%> (ø)
src/wp-cli/util.js 100.00% <100.00%> (ø)
... and 2 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 05df546...9e7ea7e. Read the comment docs.

@OmriBarZik OmriBarZik merged commit dd73960 into main Nov 4, 2020
@OmriBarZik OmriBarZik deleted the feature/theme-site-conf branch November 4, 2020 15:47
@OmriBarZik OmriBarZik linked an issue Nov 4, 2020 that may be closed by this pull request
6 tasks
@OmriBarZik OmriBarZik removed a link to an issue Nov 4, 2020
6 tasks
@github-actions
Copy link

🎉 This PR is included in version 1.0.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants