Automattic / Automattic/jetpack
Sync Package: remove dependency on JETPACK__PLUGIN_DIR from packages
- 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
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