Skip to content
This repository was archived by the owner on Sep 30, 2026. It is now read-only.

Use the standard repository setup - #2

Closed
silveltman wants to merge 2 commits into
mainfrom
chore/standard-setup
Closed

silveltman wants to merge 2 commits into
mainfrom
chore/standard-setup

Conversation

@silveltman

@silveltman silveltman commented Sep 30, 2026 •

Copy link
Copy Markdown
Contributor

Aligns this repository with the shared setup used across fulldotdev:

  • packageManager (none) -> pnpm@11.28.0
  • add format script
  • check "astro check" -> "prettier --check . && astro check"
  • .nvmrc (none) -> 24

The organization Check runs pnpm check on every PR. Netlify only builds.

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Chores
    • Added formatting validation to the project’s checks and a command to format files automatically.
    • Updated the project’s recommended Node.js version.
    • Standardized code formatting across components and documentation; these changes do not alter app behavior.

silveltman and others added 2 commits September 30, 2026 21:07
- packageManager (none) -> pnpm@11.28.0
- add format script
- check "astro check" -> "prettier --check . && astro check"
- .nvmrc (none) -> 24

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Sep 30, 2026

Copy link
Copy Markdown

Review in Change Stack →

Navigate logical layers of code changes, visualize relationships, and explore their blast radius.

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Advanced

Run ID: a559afb0-b12a-424a-8e51-ef12e07febf6

📥 Commits

Reviewing files that changed from the base of the PR and between e8a4f03 and a60ca6e.

📒 Files selected for processing (23)
  • .nvmrc
  • AGENTS.md
  • package.json
  • src/components/blocks/cta-8.astro
  • src/components/blocks/hero-12.astro
  • src/components/blocks/hero-13.astro
  • src/components/blocks/pricings-1.astro
  • src/components/blocks/pricings-3.astro
  • src/components/ui/footer/footer-copyright.astro
  • src/components/ui/native-select/native-select-option.astro
  • src/components/ui/native-select/native-select.astro
  • src/components/ui/table/index.ts
  • src/components/ui/table/table-body.astro
  • src/components/ui/table/table-caption.astro
  • src/components/ui/table/table-cell.astro
  • src/components/ui/table/table-footer.astro
  • src/components/ui/table/table-head.astro
  • src/components/ui/table/table-row.astro
  • src/components/ui/tabs/index.ts
  • src/components/ui/tabs/tabs-content.astro
  • src/components/ui/tabs/tabs-trigger.astro
  • src/components/ui/tabs/tabs.astro
  • src/schemas/blocks/pricings-embed-1.ts
💤 Files with no reviewable changes (13)
  • src/components/ui/table/table-row.astro
  • src/components/ui/tabs/tabs-content.astro
  • src/components/ui/table/table-head.astro
  • src/components/ui/tabs/index.ts
  • src/components/ui/native-select/native-select-option.astro
  • src/components/ui/table/table-body.astro
  • src/components/ui/tabs/tabs.astro
  • src/components/ui/tabs/tabs-trigger.astro
  • src/components/ui/table/index.ts
  • src/components/ui/native-select/native-select.astro
  • src/components/ui/table/table-caption.astro
  • src/components/ui/table/table-cell.astro
  • src/components/ui/table/table-footer.astro

Limit details: You’ve used all 10 included reviews currently available.


📝 Walkthrough

Walkthrough

The changes set Node.js to version 24 and declare pnpm as the package manager. The check script now runs Prettier before astro check, and a format script runs Prettier in write mode. Documentation and source files also receive formatting edits. The supplied summaries report no component behavior or inferred type changes.

Priority: ⬇️ Low

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

Change: Other

Merge Risk: ⚪ Minimal · up to a60ca

The changes update developer tooling and formatting without an established impact on application behavior. No actionable merge-blocking risk is evident.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and concisely describes the repository setup changes.
Description check ✅ Passed The description directly explains the package manager, formatting, check script, and Node.js version changes.
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.
✨ Finishing Touches 💡 1
🛠️ Fix failing CI checks 💡
  • Commit to this branch
  • Create a new PR
📝 Generate docstrings
  • Commit to this branch
  • Create a new PR
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR
  • Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Autopilot is currently an internal CodeRabbit preview.

Usage-based review receipt

  • Mode: Continue automatically
  • Reviewed files: 10
  • Waived: $2.50 (charged $0.00)
  • View usage details

Note

This review exceeded your plan’s limits and used usage-based reviews—free during trial, billed after paid activation unless disabled. Manage usage-based reviews.


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

@silveltman

Copy link
Copy Markdown
Contributor Author

Closing: this repository is being archived.

@silveltman silveltman closed this Sep 30, 2026
@silveltman
silveltman deleted the chore/standard-setup branch September 30, 2026 20:20
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant