Skip to content

GHCP - Run: Ex3 Multi-File Refactor - #33

Open
DWSPalmer wants to merge 34 commits into
mainfrom
run/DWSPalmer/ex3-multifile-refactor
Open

GHCP - Run: Ex3 Multi-File Refactor#33
DWSPalmer wants to merge 34 commits into
mainfrom
run/DWSPalmer/ex3-multifile-refactor

Conversation

@DWSPalmer

Copy link
Copy Markdown
Owner

Title: GHCP - Run: Ex3 Multi-File Refactor

Summary

  • Added shared observability helper module samples/book-app-project/observability.py
  • Refactored samples/book-app-project/utils.py and samples/book-app-project/books.py to use shared emit/toggle helpers
  • Added run-level refactor notes in ai-track-docs/run-refactor-ex3.md

Evidence

  • Validation command:
    • cd samples/book-app-project
    • . .venv/bin/activate
    • python -m pytest tests/test_utils.py tests/test_books.py -q
  • Key output:
    • 20 passed in 0.10s

Risk and Rollback

  • Risk: low
  • Rollback: revert commit fa01ef8

Track

  • Level: Run
  • Exercise: Ex3

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