Per-demo runnable scripts + Makefile targets#2
Open
noahgift wants to merge 1 commit into
Open
Conversation
Split the demo surface into four standalone scripts so each can be run,
filmed, or screen-shared independently:
scripts/demo-1-fundamentals.sh sql/01-fundamentals/*.sql (5 files)
scripts/demo-2-joins.sh sql/02-joins/*.sql (3 files)
scripts/demo-3-pagila.sh sql/pagila-analytics/*.sql (3 files)
scripts/demo-4-rust.sh Rust capstone, 3 contract-enforced reports
(accepts a limit: demo-4-rust.sh 5)
scripts/demo-all.sh run all four in sequence
Each script:
- starts postgres + loads Pagila if not already up
- prints a cyan banner per step ("=== <what> ===")
- exits non-zero on any failure (set -euo pipefail)
- demo-4 writes JSON to out/ — added to .gitignore
Makefile targets mirror each script: make demo-1-fundamentals … make demo-all.
Help target updated with the new targets.
Also adds a 4-line header to each pagila-analytics/*.sql query explaining
its purpose and tying it to the corresponding postgres-reports Rust function.
cc bumped 1.2.61 -> 1.2.62 (cargo update -p cc) so fresh clones build
cleanly under Rust 1.95 without the upstream apple_sdk_name regression.
cargo test --release: 13/13 pass against the dockerised Postgres 16.
Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
scripts/demo-1-fundamentals.sh— sql/01-fundamentals/*.sql (5 files)scripts/demo-2-joins.sh— sql/02-joins/*.sql (3 files)scripts/demo-3-pagila.sh— sql/pagila-analytics/*.sql (3 files)scripts/demo-4-rust.sh [LIMIT]— Rust capstone, 3 contract-enforced reportsscripts/demo-all.sh— runs every demo in sequenceset -euo pipefail).make demo-1-fundamentals…make demo-all.sql/pagila-analytics/*.sqlquery.cc1.2.61 → 1.2.62 inCargo.lockso fresh clones build cleanly under Rust 1.95./out/and.idea/to.gitignore(demo-4 writes JSON toout/).Test plan
bash scripts/demo-1-fundamentals.sh— 5 SQL files run cleanlybash scripts/demo-2-joins.sh— 3 files run cleanlybash scripts/demo-3-pagila.sh— 3 reports, 10 rows eachbash scripts/demo-4-rust.sh 3— 3 JSON files written toout/, contracts passcargo test --release— 13/13 pass against dockerised Postgres 16cargo llvm-cov --workspace --summary-only— 100% lines / 100% functions on both lib.rs and main.rs🤖 Generated with Claude Code