Skip to content

fix: add missing checkout in frontend workflow#42

Merged
G36maid merged 1 commit into
mainfrom
fix/frontend-checkout
Feb 15, 2026
Merged

fix: add missing checkout in frontend workflow#42
G36maid merged 1 commit into
mainfrom
fix/frontend-checkout

Conversation

@G36maid

@G36maid G36maid commented Feb 15, 2026

Copy link
Copy Markdown
Contributor

目的

Prettier 設定
jpcorrect-frontend 根目錄下有 .prettierrc。
判斷邏輯:共用的 setup-prettier.sh 會檢查 if [ ! -f ".prettierrc" ] ...。
結果:腳本會發現 .prettierrc 存在,因此輸出 Prettier config already exists, skipping creation.,並使用注入的配置。

ESLint 設定
jpcorrect-frontend 根目錄下有 eslint.config.mjs。
判斷邏輯:共用的 setup-eslint.sh 會檢查 ... && [ ! -f "eslint.config.mjs" ]。
結果:腳本會發現 eslint.config.mjs 存在,因此跳過預設注入,使用你注入的配置。

方法/實作說明

補上缺失的 frontend checkout
其他 workflow 都已經正常運作,只有 frontend 缺失

關聯 Issue

fix #41

附註

  • TODO:
    • 確認 frontend ci 成功運作

- add missing checkout in reusables frontend workflow

fix #41
@G36maid G36maid force-pushed the fix/frontend-checkout branch from c49ccea to 4833c9f Compare February 15, 2026 13:36
@github-actions

Copy link
Copy Markdown

🛡️ PR Quality Check Summary

PR Title: Passed (Length: 46/75, Format: OK). fix: add missing checkout in frontend workflow
Branch Name: Follows naming convention (fix/frontend-checkout)
Commit Messages: All 1 commit(s) passed (Length, Format, Case)
Conflicts: No merge conflict markers found
YAML Files: All 12 file(s) passed validation
JSON Files: All 1 file(s) are valid
TOML Files: No TOML files found to check


🎉 All checks passed!

@torrid-fish torrid-fish left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@G36maid G36maid added this pull request to the merge queue Feb 15, 2026
Merged via the queue into main with commit fc0a6bc Feb 15, 2026
6 checks passed
@G36maid G36maid deleted the fix/frontend-checkout branch February 15, 2026 14:11
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.

frontend missing checkout

2 participants