Automattic / Automattic/jetpack

Sync Package: remove dependency on JETPACK__PLUGIN_DIR from packages

Open
#12,739 1 comment 0 reactions 0 assignees View on GitHub
[Focus] Jetpack DNA [Package] Sync
Dominant language
PHP
Stars
1.8k
Forks
898
Avg merge
1d 18h
Merged PRs (30d)
774

Description

See:
https://github.com/Automattic/jetpack/blob/14a647f75f1acaabac06af78ba0fb46a7fe36262/packages/sync/legacy/class.jetpack-sync-module-options.php#L156

Packaged classes should not require nor depend on code from jetpack.

This issue applies to:

- Automattic\Jetpack\JITM
- Automattic\Jetpack\Sync\Defaults
- Automattic\Jetpack\Sync\Functions
- Automattic\Jetpack\Sync\Listener
- Automattic\Jetpack\Sync\Modules\Options

Contributor guide

Open the contributing guide

Research direction

Start with packages/sync/legacy/class.jetpack-sync-module-options.php at the referenced line, then trace JETPACK__PLUGIN_DIR usage in Automattic\Jetpack\JITM, Sync\Defaults, Sync\Functions, Sync\Listener, and Sync\Modules\Options. Done means these packaged classes no longer require or depend on code from Jetpack.

Written by the indexing model from the issue text.

Assessment

Tech stack
php
Domain
backend
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.