Skip to content

Add ci: GitHub Actions workflow#256

Merged
boojack merged 1 commit intomainfrom
add-github-action-ci
Mar 25, 2026
Merged

Add ci: GitHub Actions workflow#256
boojack merged 1 commit intomainfrom
add-github-action-ci

Conversation

@memoclaw
Copy link
Copy Markdown
Contributor

@memoclaw memoclaw commented Mar 25, 2026

Summary

  • add a GitHub Actions CI workflow for push and pull_request
  • split validation into separate lint and build jobs with shared Node.js and pnpm setup
  • install with --ignore-scripts and run MDX generation explicitly before build to keep CI deterministic
  • include the minimal Biome-driven cleanup needed for the new lint job to pass

Testing

  • pnpm lint
  • pnpm build

Summary by CodeRabbit

  • Chores
    • Added automated linting and build verification via CI pipeline
    • Standardized documentation metadata formatting
    • Improved code style consistency with type-only import statements

@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Mar 25, 2026

Caution

Review failed

Pull request was closed or merged during review

📝 Walkthrough

Walkthrough

Adds a GitHub Actions CI workflow with lint and build jobs, reformats API documentation metadata JSON files from multi-line to single-line array formatting, and updates component import statements to use type-only syntax while refactoring inline style logic.

Changes

Cohort / File(s) Summary
CI Workflow Configuration
.github/workflows/ci.yml
Added new GitHub Actions workflow triggering on push and pull request events. Includes lint job (runs pnpm lint) and build job (runs fumadocs-mdx generation followed by pnpm build). Configures concurrency, minimal permissions, and disables Next.js telemetry.
API Metadata Files (0-26-2)
content/docs/api/0-26-2/*/meta.json, content/docs/api/0-26-2/meta.json
Reformatted pages arrays from multi-line to single-line JSON format across Activity, Attachment, Auth, Identity Provider, Instance, Memo, Shortcut, and User services. Added trailing newlines to ensure proper file termination.
API Metadata Files (latest)
content/docs/api/latest/*/meta.json, content/docs/api/latest/meta.json
Applied same formatting changes as 0-26-2: converted multi-line pages arrays to single-line format for Attachment, Auth, Identity Provider, Instance, Memo, Shortcut, and User services. Ensured proper trailing newlines.
API Root Metadata
content/docs/api/meta.json
Condensed pages array from multi-line to single-line format and added trailing newline.
Component Imports and Refactoring
src/components/hero-section.tsx, src/components/section-header.tsx, src/components/memo-hero-mock.tsx
Updated imports to use type-only syntax for ReactNode and LucideIcon. Refactored inline ternary transform logic in memo hero mock to single-line format while preserving conditional behavior.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Poem

🐰 A bunny's delight, the workflows align,
Arrays now single, all formatted fine!
Components cleaned with imports so neat,
The CI pipeline makes building complete! ✨

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title directly and clearly describes the main change: adding a GitHub Actions CI workflow file (.github/workflows/ci.yml), which represents the primary and most substantial modification in this changeset.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ 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 add-github-action-ci

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

@boojack boojack merged commit 0db4d44 into main Mar 25, 2026
4 of 5 checks passed
@boojack boojack deleted the add-github-action-ci branch April 1, 2026 09:54
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.

2 participants