Skip to content

Feat/implement GitHub workflow#7

Merged
arxja merged 4 commits into
mainfrom
feat/implement-github-workflow
Jun 10, 2026
Merged

Feat/implement GitHub workflow#7
arxja merged 4 commits into
mainfrom
feat/implement-github-workflow

Conversation

@arxja

@arxja arxja commented Jun 10, 2026

Copy link
Copy Markdown
Owner

Summary by CodeRabbit

  • New Features

    • Established automated CI/CD pipeline with continuous testing on every push and pull request.
    • Added GitHub issue templates for bug reports and feature requests to streamline community feedback.
  • Bug Fixes

    • Fixed CSS directive formatting.
  • Chores

    • Updated test script configuration for CI environment.

@coderabbitai

coderabbitai Bot commented Jun 10, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

Warning

Review limit reached

@arxja, we couldn't start this review because you've reached your PR review rate limit.

More reviews will be available in 38 minutes and 51 seconds. Learn how PR review limits work.

Your organization has run out of usage credits. Purchase more in the billing tab.

⌛ How to resolve this issue?

After more reviews become available, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans include higher PR review limits than trial, open-source, and free plans. In all cases, reviews become available again over time. During sustained high-volume PR review activity, CodeRabbit may temporarily slow when the next review becomes available.

Please see our Fair Usage Limits Policy for further information.

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 64ab2ddd-c9af-4bb5-8753-1b7828b851b9

📥 Commits

Reviewing files that changed from the base of the PR and between 73dbdd3 and f71f1d4.

📒 Files selected for processing (3)
  • .github/ISSUE_TEMPLATE/bug_report.md
  • .github/ISSUE_TEMPLATE/feature_request.md
  • .github/workflows/ci.yml
📝 Walkthrough

Walkthrough

This PR establishes foundational project infrastructure by adding GitHub issue templates for standardized bug reports and feature requests, configuring a GitHub Actions CI workflow that installs dependencies with pnpm and runs tests and builds, updating the test:ci script with a Vitest flag, and fixing a Tailwind CSS syntax error.

Changes

Project Infrastructure & CI

Layer / File(s) Summary
GitHub Issue Templates
.github/ISSUE_TEMPLATE/bug_report.md, .github/ISSUE_TEMPLATE/feature_requset.md
Bug report template with sections for description, reproduction steps, expected behavior, screenshots, and environment. Feature request template with problem statement, desired solution, alternatives, and affected-area checklist.
CI Workflow and Test Configuration
.github/workflows/ci.yml, package.json
CI workflow triggers on push and pull request to main, runs on Node.js 24 with pnpm v11, installs dependencies with frozen lockfile, and executes test:ci and build jobs. The test:ci script is updated with --passWithNoTests for Vitest.
Tailwind @apply Directive Fix
app/globals.css
The html rule's Tailwind @apply directive is corrected by removing stray backticks to enable proper CSS parsing.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

  • arxja/saasify#1: The test:ci npm script update in this PR directly extends Vitest setup that was introduced in the related PR.

Poem

🐰 Fresh workflows sprouting, templates bloom,
Bug reports and features find their room,
Pnpm caches dance so fine,
CSS backticks? Now aligned!
The project grows, organized and bright! ✨

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'Feat/implement GitHub workflow' partially describes the changeset, focusing on the CI workflow addition but omitting the bug report template, feature request template, and CSS fix that are also included.
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.

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

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feat/implement-github-workflow

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 and usage tips.

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 5

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In @.github/ISSUE_TEMPLATE/bug_report.md:
- Line 32: Rename the ambiguous "BugOn" field label in the bug report template
to a clearer term (e.g., "Bug Location" or "Affected Component") by replacing
the "BugOn: [feature/core/etc]" string with the chosen label and corresponding
placeholder; also update any tests, docs, or references that expect "BugOn" so
they use the new label to keep everything consistent.

In @.github/ISSUE_TEMPLATE/feature_requset.md:
- Around line 1-25: The filename in the repo contains a typo: rename the
ISSUE_TEMPLATE file named feature_requset.md to feature_request.md; update any
references or links inside the repository that point to
".github/ISSUE_TEMPLATE/feature_requset.md" so they now point to
".github/ISSUE_TEMPLATE/feature_request.md" (ensure the new filename is
committed and the old file removed/renamed in the same commit to preserve
history).

In @.github/workflows/ci.yml:
- Line 16: The checkout step currently uses actions/checkout@v4 without
disabling credential persistence; update the checkout step (the
actions/checkout@v4 invocation) to include persist-credentials: false so the
GITHUB_TOKEN is not written into .git/config and cannot be leaked to later steps
or artifacts.
- Around line 10-27: The workflow job "test" currently relies on default (broad)
permissions—add an explicit minimal permissions block under the job to follow
least-privilege; update the "test" job in .github/workflows/ci.yml to include a
permissions mapping (for this CI that only checks out code, installs and runs
tests/builds you typically only need at minimum: contents: read) and remove
reliance on defaults so the job declares permissions: contents: read (add other
specific permissions only if your steps require them).
- Around line 16-20: The workflow currently uses mutable tags for actions
(actions/checkout@v4, pnpm/action-setup@v2, actions/setup-node@v4); replace each
tag reference with the corresponding immutable commit SHA for that action (e.g.,
actions/checkout@<commit-sha>) by looking up the action's repo to find the exact
commit for the desired release and update the .github/workflows/ci.yml entries
accordingly so the workflow is pinned to specific commit SHAs.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 6fe7ef5a-b35d-4048-806c-c77d25c76918

📥 Commits

Reviewing files that changed from the base of the PR and between 8f8aad5 and 73dbdd3.

📒 Files selected for processing (5)
  • .github/ISSUE_TEMPLATE/bug_report.md
  • .github/ISSUE_TEMPLATE/feature_requset.md
  • .github/workflows/ci.yml
  • app/globals.css
  • package.json

Comment thread .github/ISSUE_TEMPLATE/bug_report.md Outdated
Comment thread .github/ISSUE_TEMPLATE/feature_requset.md Outdated
Comment thread .github/workflows/ci.yml Outdated
Comment thread .github/workflows/ci.yml Outdated
Comment thread .github/workflows/ci.yml Outdated
@arxja arxja merged commit d7346d4 into main Jun 10, 2026
2 checks passed
@arxja arxja deleted the feat/implement-github-workflow branch June 10, 2026 07:09
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