Skip to content

docs: make execution report example portable - #273

Merged
Pigbibi merged 2 commits into
mainfrom
codex/schwab-env-example-portability-20260716
Jul 16, 2026
Merged

docs: make execution report example portable#273
Pigbibi merged 2 commits into
mainfrom
codex/schwab-env-example-portability-20260716

Conversation

@Pigbibi

@Pigbibi Pigbibi commented Jul 16, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • replace the deployment-specific execution-report prefix in .env.example with an explicit public GCS placeholder
  • add one focused contract test for the exact placeholder and URI structure
  • preserve every other .env.example line unchanged

Scope

.env.example and its focused test only. No workflow, runtime, verifier, permission, trigger, dependency, deployment, or live behavior changes.

Validation

  • python3 -m pytest -q tests/test_env_example.py (1 passed)
  • python3 -m ruff check .
  • python3 -m compileall -q .
  • non-target .env.example identity check
  • git diff HEAD^ --check

Full suite result: 2 failed, 178 passed. The same two failures existed on the clean baseline and are unrelated to this sample-only change:

  • tests/test_shared_chat_id_fallback.py::SharedChatIdTests::test_global_telegram_chat_id_is_used
  • tests/test_uv_dependency_workflow.py::test_ci_and_docker_use_uv_lock

Risk

Low. The example file is not loaded by tracked runtime or workflow code; actual execution-report configuration continues to come from runtime environment or repository variables.

Co-Authored-By: Codex <noreply@openai.com>
@github-actions

github-actions Bot commented Jul 16, 2026

Copy link
Copy Markdown

🤖 Codex PR Review

Merge allowed: No issues found


Review by Codex PR Review bot • PR

…ple-portability-20260716

Co-Authored-By: Codex <noreply@openai.com>
@Pigbibi
Pigbibi merged commit e652bd4 into main Jul 16, 2026
3 checks passed
@Pigbibi
Pigbibi deleted the codex/schwab-env-example-portability-20260716 branch July 16, 2026 01:04
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