CDP-27 chore⚙️: GitHub Actions (CI) 세팅 #1
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
Codex Review: Here are some suggestions.
Reply with @codex fix comments to fix any unresolved comments.
About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you open a pull request for review, mark a draft as ready, or comment "@codex review". If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex fix this CI failure" or "@codex address that feedback".
| - run: pnpm install --frozen-lockfile | ||
| - run: pnpm run lint --if-present |
There was a problem hiding this comment.
[P0] Lint job fails due to misplaced --if-present flag
The lint job calls pnpm run lint --if-present, but in pnpm the --if-present option must precede the script name (pnpm run --if-present lint). As written, pnpm executes the lint script and forwards --if-present to eslint, which rejects the unknown flag and exits with code 2. This causes every CI run to fail even when linting succeeds locally.
Useful? React with 👍 / 👎.
| - run: pnpm install --frozen-lockfile | ||
| - run: pnpm run test --if-present -- --ci |
There was a problem hiding this comment.
[P0] Test job cannot skip missing script
The test step also places --if-present after the script (pnpm run test --if-present -- --ci). Because pnpm treats --if-present as an argument to the script, the command fails with Missing script: test when no test script is defined, so the workflow errors even when tests are intentionally absent. To make the step optional, the flag needs to appear before the script name or the step should handle the absence explicitly.
Useful? React with 👍 / 👎.
No description provided.