Conversation
|
Warning Review limit reachedNext included review available in 34 minutes. View limit detailsLimit details: You’ve used the included review currently available. You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. Review configuration: ⚙️ Run configurationConfiguration used: Repository: lightspeedwp/.github/.coderabbit.yml Review profile: CHILL Plan: Advanced Run ID: 📒 Files selected for processing (3)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
📋 Changelog Quality ValidationSummary
Status✅ Validation PASSED - No new failures introduced by this PR. No action required. |
PR Template RoutingBranch Type: This PR was automatically routed based on the branch naming strategy. |
|
Tick the box to add this pull request to the merge queue (same as
|
|
Heads-up: this PR body does not include the sections from its routed template (docs/ → pr_feature.md). Missing: |
krugazul
left a comment
There was a problem hiding this comment.
@eleshar we will no longer be using the starter plugin and theme. We are generating our plugins and themes from the following 2 tools.
https://github.com/lightspeedwp/block-plugin-scaffold
https://github.com/lightspeedwp/block-theme-scaffold
Summary
Implements issue #1396: single versioned document holding the plugin advisories (
docs/PLUGIN_ADVISORIES.md, review owner + last-reviewed date in header).Honesty notes (verified, not assumed)
ls-starter-plugin/inc/). Everything else is marked planned with the tracking issue where one exists (SVG → ls-starter-plugin#3, mail sender → ls-starter-plugin#2) or an explicit 'no tracking issue yet' where none does. The issue text describes the target state; the doc does not claim it as current.phpmailer_initfires after coresetFrom();@view-transitionis not Baseline (progressive enhancement only);gform_disable_cssbreaks conditional logic per GF docs.Acceptance criteria mapping
Verification