Skip to content

Add GitHub Actions CI for /website#51

Merged
em3s merged 1 commit intokakao:mainfrom
csbiy:add-website-ci
Jan 18, 2026
Merged

Add GitHub Actions CI for /website#51
em3s merged 1 commit intokakao:mainfrom
csbiy:add-website-ci

Conversation

@csbiy
Copy link
Copy Markdown
Contributor

@csbiy csbiy commented Jan 17, 2026

Summary

This PR adds CI coverage for the /website documentation site.

Changes

  • Added .github/workflows/website-ci.yml
  • Runs ESLint and Prettier checks on pull requests
  • Uses ESLint/Prettier config from Add eslint and prettier config #27
  • Triggers only when:
    • Files under /website change
    • The workflow file itself changes

Testing

  • Local ESLint check passed (no errors)
  • Local Prettier check passed (all files formatted correctly)
  • Workflow syntax validated

Implementation Details

Workflow file: .github/workflows/website-ci.yml

Trigger: Pull requests with path filter

  • website/**
  • .github/workflows/website-ci.yml

Jobs:

  1. Checkout code
  2. Setup Node.js 20 with npm cache
  3. Install dependencies (npm ci)
  4. Run ESLint (npm run lint)
  5. Check Prettier formatting (npm run format:check)

Closes #47

@csbiy csbiy requested a review from em3s as a code owner January 17, 2026 09:27
@dosubot dosubot bot added size:M This PR changes 30-99 lines, ignoring generated files. module:website labels Jan 17, 2026
@csbiy csbiy force-pushed the add-website-ci branch 2 times, most recently from 090de79 to 782893d Compare January 17, 2026 09:38
@csbiy
Add GitHub Actions CI for /website
0e4d5f7 
- Run ESLint and Prettier on pull requests
- Build website to verify it compiles successfully
- Trigger only when website files or workflow changes
- Use config from kakao#27

Closes kakao#47
@em3s
Copy link
Copy Markdown
Contributor

em3s commented Jan 18, 2026

LGTM 👍

Reviewed the workflow and it looks correct.
We’ll follow up post-merge if needed.

Thanks for your continued contributions!

@csbiy

@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Jan 18, 2026
@em3s em3s merged commit 0e21a13 into kakao:main Jan 18, 2026
3 checks passed
@em3s em3s mentioned this pull request Jan 18, 2026
Merged
@em3s
Copy link
Copy Markdown
Contributor

em3s commented Jan 18, 2026

After merge, I think it makes sense to run the CI on main and skip draft PRs, to align with the main CI workflow.
I’ll follow up with a separate PR #54. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@em3s em3s em3s approved these changes

Assignees

@csbiy csbiy

Labels

lgtm This PR has been approved by a maintainer size:M This PR changes 30-99 lines, ignoring generated files.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Add GitHub Actions CI for /website

2 participants

@csbiy @em3s