Skip to content

docs: add demo video and build-notify example#25

Merged
chsm04 merged 1 commit intomainfrom
docs/demo-video-and-build-notify
Mar 25, 2026
Merged

docs: add demo video and build-notify example#25
chsm04 merged 1 commit intomainfrom
docs/demo-video-and-build-notify

Conversation

@chsm04
Copy link
Copy Markdown
Owner

@chsm04 chsm04 commented Mar 25, 2026

Summary

  • 데모 영상을 build-notify 예시로 교체 (이메일 노출 영상 제거)
  • examples/build-notify.sh 추가
  • Build Error 섹션에 영상 삽입

Test plan

  • build-notify.sh 동작 확인 완료

🤖 Generated with Claude Code

- Replace demo video with build-notify example (no email exposure)
- Add examples/build-notify.sh — build wrapper that sends errors to Pulse
- Add build-notify video to Build Error section in README

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@chsm04 chsm04 force-pushed the docs/demo-video-and-build-notify branch from d68b7f0 to 178a8a2 Compare March 25, 2026 09:12
@chsm04 chsm04 merged commit a88ded0 into main Mar 25, 2026
1 check passed
@chatgpt-codex-connector
Copy link
Copy Markdown

💡 Codex Review

OUTPUT=$("$@" 2>&1) && exit 0

P2 Badge Preserve wrapped command output on successful builds

The wrapper captures all stdout/stderr in OUTPUT=$("$@" 2>&1) and immediately exits on success, so any successful build produces no visible logs. In CI or local automation this hides normal build output entirely, which undermines diagnostics and can make the step appear to do nothing. Please avoid swallowing output on success (for example by streaming it or replaying it before exiting).

ℹ️ 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
  • 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 address that feedback".

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