Skip to content

feat: remove deprecated components and update changelog with recent changes - #125

Merged
ditahkk merged 2 commits into
mainfrom
issues/121-instrument-follow-ups
Oct 12, 2026
Merged

ditahkk merged 2 commits into
mainfrom
issues/121-instrument-follow-ups

Conversation

@ditahkk

@ditahkk ditahkk commented Oct 12, 2026 •

Copy link
Copy Markdown
Contributor

What changed

Why


Summary by CodeRabbit

  • Documentation
    • Updated the French changelog with product and provisioning changes, including ZCP Store, instance passwords, separate compute and root-storage plans, and CLI and Terraform/OpenTofu releases.
    • Added French titles to selected changelog entries.
  • Removals
    • Removed the principle quote panel, tutorial cards, and PeerPush badge and recognition content.

@coderabbitai

coderabbitai Bot commented Oct 12, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration
  • Configuration used: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: a7b2a096-f911-4ed6-a6a9-b6af79f28f72

📥 Commits

Reviewing files that changed from the base of the PR and between fe83af3 and 9fdb0f7.


📒 Files selected for processing (6)
  • src/components/PrinciplePanel.astro
  • src/components/TutorialCards.astro
  • src/content/docs/fr/changelog/index.md
  • src/data/changelog-feed.ts
  • src/data/peerpush.ts
  • src/styles/custom.css

💤 Files with no reviewable changes (4)
  • src/components/PrinciplePanel.astro
  • src/styles/custom.css
  • src/components/TutorialCards.astro
  • src/data/peerpush.ts

Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review.



📝 Walkthrough

Walkthrough

The pull request removes the principle panel and tutorial card components, adds French changelog entries and feed titles, and removes PeerPush URL exports and related CSS.

Changes

Principle panel removal

Layer / File(s) Summary
Remove the principle panel
src/components/PrinciplePanel.astro
The localized quote, pillar markup, and associated styles were removed.

Tutorial cards removal

Layer / File(s) Summary
Remove tutorial cards
src/components/TutorialCards.astro
The tutorial selection, sorting, card rendering, and description clamp were removed.

French changelog additions

Layer / File(s) Summary
Product updates and feed titles
src/content/docs/fr/changelog/index.md, src/data/changelog-feed.ts
The French changelog adds entries about ZCP Store, instance passwords, and compute and storage separation. The feed adds French titles for new and existing entries.
CLI release notes
src/content/docs/fr/changelog/index.md
French entries describe CLI releases 1.0.1, 1.0.0, and 0.0.31.
Provider release notes
src/content/docs/fr/changelog/index.md
French entries describe provider releases 1.0.0 and 0.3.0.

PeerPush removal

Layer / File(s) Summary
Remove PeerPush URLs and styles
src/data/peerpush.ts, src/styles/custom.css
The PeerPush listing and badge URL exports and related card styles were removed.

Priority: ⬇️ Low

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

Change: Other


Merge Risk: ⚪ Minimal · up to 9fdb0

No actionable issue remains: the omitted release is consistent with the selective recent list, and the removed components and PeerPush assets have no remaining tracked consumers.

Pre-merge checks | Passed 4 | Failed 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check Warning The description includes the required headings, but both sections remain blank and do not explain the changes or rationale. Add a concise summary of the removed components, changelog updates, and language indications. Explain why these changes were made, and link the related issue if available.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check Passed The title clearly summarizes the main changes: removal of deprecated components and changelog updates.
Docstring Coverage Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 1…
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.

  • Fix all pre-merge checks with AI
✨ Finishing Touches
📝 Generate docstrings
  • Commit to this branch
  • Create a new PR

🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR

  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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.

❤️ Share

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

@ditahkk
ditahkk merged commit 372fa7c into main Oct 12, 2026
5 checks passed
@ditahkk
ditahkk deleted the issues/121-instrument-follow-ups branch October 12, 2026 13:40
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