Skip to content

docs(drupal): sync playground coverage#341

Open
mberlofa wants to merge 1 commit into
mainfrom
docs/drupal-template-standards-sync
Open

docs(drupal): sync playground coverage#341
mberlofa wants to merge 1 commit into
mainfrom
docs/drupal-template-standards-sync

Conversation

@mberlofa

@mberlofa mberlofa commented Jul 3, 2026

Copy link
Copy Markdown
Contributor

Summary

  • add drupal to the playground coverage allowlist required by the chart site-sync gate

Validation

  • make site-sync-check CHART=drupal
  • npm run lint
  • npm run format:check
  • npm run build
  • make release-check REPO=site
  • make attribution-check REPO=site

Chart PR: helmforgedev/charts#662
Issue: helmforgedev/charts#633

@coderabbitai

coderabbitai Bot commented Jul 3, 2026

Copy link
Copy Markdown

Review Change Stack

📝 Walkthrough

Walkthrough

Adds a drupal entry to the siteSyncPlaygroundConfigs slug map in src/pages/playground.astro, mapping it to src/data/playground-configs.ts to include it in playback/config synchronization logic.

Changes

Playground Config Sync

Layer / File(s) Summary
Add drupal slug mapping
src/pages/playground.astro
Adds a drupal entry to siteSyncPlaygroundConfigs, mapping it to src/data/playground-configs.ts.

Estimated code review effort: 1 (Trivial) | ~2 minutes

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Title check ✅ Passed The title is concise and accurately reflects adding Drupal to playground coverage sync.
Description check ✅ Passed The description covers the change summary and validation steps, and is mostly aligned with the template.
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch docs/drupal-template-standards-sync

Comment @coderabbitai help to get the list of available commands.

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