Skip to content

docs(hoppscotch): sync template standards updates#353

Open
mberlofa wants to merge 2 commits into
mainfrom
docs/hoppscotch-template-standards-sync
Open

docs(hoppscotch): sync template standards updates#353
mberlofa wants to merge 2 commits into
mainfrom
docs/hoppscotch-template-standards-sync

Conversation

@mberlofa

@mberlofa mberlofa commented Jul 3, 2026

Copy link
Copy Markdown
Contributor

Summary

  • sync Hoppscotch docs for the template-standards chart update
  • document networkPolicy.extraEgress and extraManifests
  • correct the documented image tag to 2026.5.0
  • add Hoppscotch to the playground site-sync registry and expose network policy controls

Validation

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

Chart PR: helmforgedev/charts#675

Summary by CodeRabbit

  • New Features

    • Added Hoppscotch to the playground chart list, making it available when eligible.
    • Introduced enhanced Network Policy settings with an additional egress option to customize destination CIDRs and ports (when enabled).
  • Documentation

    • Updated Hoppscotch Helm chart docs to reference the newer image tag and documented the new extraEgress and extraManifests parameters.

@coderabbitai

coderabbitai Bot commented Jul 3, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: ffc6ef80-2af3-46d1-9055-033853b0008b

📥 Commits

Reviewing files that changed from the base of the PR and between 0a29446 and 99dcbdb.

📒 Files selected for processing (1)
  • src/pages/docs/charts/hoppscotch.mdx

📝 Walkthrough

Walkthrough

This PR adds a Network Policy configuration section to the Hoppscotch chart playground config, registers it in the playground mapping, and updates the Hoppscotch chart docs with a newer image tag and additional parameters.

Changes

Hoppscotch Network Policy playground and docs

Layer / File(s) Summary
Network Policy playground config and wiring
src/data/playground-configs.ts, src/pages/playground.astro
A collapsible Network Policy section gated by networkPolicy.enabled is added with extra egress CIDR and port fields, and the hoppscotch config is registered in the site-sync playground mapping.
Chart documentation updates
src/pages/docs/charts/hoppscotch.mdx
The referenced image tag is updated to 2026.5.0, and the Parameters table adds networkPolicy.extraEgress and extraManifests with their defaults.

Estimated code review effort: 2 (Simple) | ~10 minutes

Possibly related PRs

  • helmforgedev/site#322: Also adds a chart slug to src/pages/playground.astro’s siteSyncPlaygroundConfigs mapping.
  • helmforgedev/site#330: Also extends the same playground registry in src/pages/playground.astro with another chart slug.
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title is concise and matches the Hoppscotch docs sync and playground registry update.
Description check ✅ Passed The summary and validation sections cover the main changes; only the template's exact checkbox-style Quality Gates and Related Issue fields are missing.
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.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch docs/hoppscotch-template-standards-sync

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

@mberlofa

mberlofa commented Jul 5, 2026

Copy link
Copy Markdown
Contributor Author

Synced this site PR with the Hoppscotch chart change in helmforgedev/charts#675.

What changed:

  • Updated the Hoppscotch values reference to document that extraManifests are rendered with Helm tpl.

Validation:

  • make site-sync-check CHART=hoppscotch reports the required artifacts as OK.
  • npm run format:check passed.
  • npm run lint passed.
  • npm run build passed.
  • make release-check REPO=site passed.
  • make attribution-check REPO=site passed.

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