Skip to content

feat: DB_SCHEMA schema isolation + ACR CI secrets docs#9

Merged
lassebenni merged 2 commits into
mainfrom
feat/schema-isolation
Jun 11, 2026
Merged

feat: DB_SCHEMA schema isolation + ACR CI secrets docs#9
lassebenni merged 2 commits into
mainfrom
feat/schema-isolation

Conversation

@lassebenni

Copy link
Copy Markdown
Collaborator

Summary

  • storage.py: creates schema and sets search_path via DB_SCHEMA env var so each student's tables land in their own schema, not public
  • .env.example: adds DB_SCHEMA=dev_<your_github_handle> placeholder with explanation
  • README.md: adds DB_SCHEMA to run/deploy steps, uses <your_table> placeholder in the verify command (not weather_readings), adds "Enable ACR push from CI (optional)" section with secret names

Test plan

  • uv run python -m src.pipeline with DB_SCHEMA=dev_test creates dev_test.weather_readings
  • Without DB_SCHEMA, falls back to public (backwards-compatible)
  • README <your_table> placeholder does not confuse students into thinking the table must be named weather_readings

🤖 Generated with Claude Code

Lasse Benninga and others added 2 commits June 11, 2026 11:37
- storage.py: create schema + SET search_path using DB_SCHEMA env var
- .env.example: add DB_SCHEMA placeholder with explanation
- README.md: add DB_SCHEMA to run/deploy steps, <your_table> placeholder
  in verify command, and new Enable ACR push from CI section

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@lassebenni lassebenni merged commit 30eaf53 into main Jun 11, 2026
4 checks passed
@lassebenni lassebenni deleted the feat/schema-isolation branch June 11, 2026 09:42
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