From 09e263e3a97562916aaa36a598bd6f2b820b1c55 Mon Sep 17 00:00:00 2001 From: seonghobae <8172694+seonghobae@users.noreply.github.com> Date: Fri, 3 Jul 2026 05:10:45 +0000 Subject: [PATCH] =?UTF-8?q?=F0=9F=8E=A8=20Palette:=20=EA=B2=BD=EA=B3=A0=20?= =?UTF-8?q?=EB=A9=94=EC=8B=9C=EC=A7=80=EC=97=90=20=EC=9D=B4=EB=AA=A8?= =?UTF-8?q?=EC=A7=80=20=EC=B6=94=EA=B0=80?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit appguardrail_core/org_bundle.py의 PR 수집 건너뜀 경고 메시지에 시각적 구분을 위한 ⚠️ 이모지를 추가하고, 이에 맞춰 tests/test_org_readiness_report_script.py의 테스트 기대값을 수정했습니다. --- .Jules/palette.md | 1 + appguardrail_core/org_bundle.py | 2 +- tests/test_org_readiness_report_script.py | 2 +- 3 files changed, 3 insertions(+), 2 deletions(-) diff --git a/.Jules/palette.md b/.Jules/palette.md index f8c1fbd..9296c26 100644 --- a/.Jules/palette.md +++ b/.Jules/palette.md @@ -5,3 +5,4 @@ ## 2026-06-30 - Added Emojis to CLI Output Messages **Learning:** Adding subtle emojis to informative CLI output headers (like "Created/updated files" and "Next steps") provides clearer visual cues for developers scanning long CLI output. **Action:** Always include relevant emojis in summary output text to make success and informational messages more visually distinguishable from routine command logs. +## 2026-07-06 - Added Emojis to CLI Warning Messages\n**Learning:** Just like errors, warnings in CLIs benefit from visual cues like emojis to draw attention without being as harsh as an error.\n**Action:** When adding or modifying warning messages in the CLI, use '⚠️ Warning: [Message]' format. diff --git a/appguardrail_core/org_bundle.py b/appguardrail_core/org_bundle.py index 3634d07..6022518 100644 --- a/appguardrail_core/org_bundle.py +++ b/appguardrail_core/org_bundle.py @@ -282,7 +282,7 @@ def gh_pr_list( except subprocess.CalledProcessError as exc: warning = f"Skipped PR collection for {full_name}: {gh_error_message(exc)}" warnings.append(warning) - print(f"warning: {warning}", file=sys.stderr) + print(f"⚠️ Warning: {warning}", file=sys.stderr) continue for pull in repo_pulls: pull["repository"] = {"nameWithOwner": full_name} diff --git a/tests/test_org_readiness_report_script.py b/tests/test_org_readiness_report_script.py index 506b51c..2262bcc 100644 --- a/tests/test_org_readiness_report_script.py +++ b/tests/test_org_readiness_report_script.py @@ -120,6 +120,6 @@ def fake_gh_json(args): "Skipped PR collection for ContextualWisdomLab/bad: HTTP 502: Bad Gateway" ] assert ( - "warning: Skipped PR collection for ContextualWisdomLab/bad" + "⚠️ Warning: Skipped PR collection for ContextualWisdomLab/bad" in capsys.readouterr().err )