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

feat(dev-env): write integrations config when creating an environment #2199

Open
wants to merge 1 commit into
base: trunk
Choose a base branch
from

Conversation

sjinks
Copy link
Member

@sjinks sjinks commented Jan 23, 2025

Description

This PR updates the environment creation workflow to create integration configuration and mount it into WordPress.

Note that WordPress will not load the configuration yet: this requires changes to the dev-tools and/or wordpress Docker images, as well as mu-plugins.

Related: Automattic/vip-go-mu-plugins#6101

Pull request checklist

New release checklist

Steps to Test

  1. vip @app.env dev-env create < /dev/null
  2. Observe that ~/.local/share/vip/dev-environment/<NAME>/integrations-config/integrations.json exists
  3. vip dev-env start
  4. vip dev-env shell -- cat /wp/config/integrations-config/integrations.json

@sjinks sjinks requested a review from rinatkhaziev January 23, 2025 09:56
@sjinks sjinks self-assigned this Jan 23, 2025
Copy link
Contributor

github-actions bot commented Jan 23, 2025

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

OpenSSF Scorecard

PackageVersionScoreDetails

Scanned Files

@sjinks sjinks marked this pull request as ready for review January 23, 2025 09:59
@sjinks sjinks force-pushed the write-integrations-config branch from 1adb7d1 to 889409c Compare January 23, 2025 10:03
@sjinks sjinks changed the title feat: write integrations config when creating an environment feat(dev-env): write integrations config when creating an environment Jan 23, 2025
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.

1 participant