Demo-proof datascope for client launch#6
Merged
Conversation
- Catch invalid --sheet (index/name) with friendly error listing available sheets - Fix singular/plural grammar: "1 value was" not "were"; "(1 time)" not "times" - Update argparse description to mention .parquet alongside .xlsx/.csv - Add numpy to [dev] extras so generate_sample.py works after pip install - Warn when --sheet is passed for non-Excel files (no silent ignore) All 283 tests pass. All 5 output formats verified. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- PDF title page: "datascope" branding above report title, version shown - PDF footer: version + "pip install datascope-dq" on every page - HTML: favicon (navy 'd' circle), meta description, generator meta tag - HTML footer: version + install command - JSON: "generator" field with tool name and version for provenance Reports now identify themselves as datascope products, not anonymous outputs. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- Add PyPI, CI, Python version, and license badges to README - Create CHANGELOG.md with v1.0 through v2.2 release history - Generate and commit HTML + annotated-Excel sample outputs - Update samples/README.md to reflect all output formats - Move internal dev docs (brainstorms, plans) to .dev/ directory - Regenerate PDF sample outputs with branding Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
…launch Session summary — 2026-05-16 Starting point: datascope v2.2.0 shipped on PyPI. Preparing to share demo links with first round of prospective clients (C-suite, operations). What we did: - Full 4-phase project audit (Round 2) focused on demo/client readiness - Phase 1: Baseline — v2.2 functional, 283 tests, audience is prospects - Phase 2: Internal review — 4 demo-killers, 6 professionalism issues, 5 polish - Phase 3: Landscape — niche still empty, GX acquired by Fivetran, no PDF competitor - Phase 4: Synthesized 4 moves (Demo-Proof → Brand → Repo Hygiene → Growth) - Executed Move 1: fixed --sheet crash, grammar, help text, numpy dep, sheet warning - Executed Move 2: PDF branding + footer, HTML favicon + meta, JSON generator field - Executed Move 3: badges, CHANGELOG.md, sample outputs, moved dev docs to .dev/ - Created PR #6 with all changes What worked: Audit phases built context naturally. Independent sub-tasks (A1-A4) parallelized well. Surprises: Grammar bugs only manifest with exactly 1 minority-type value (sample files have 15, so never triggered). Fivetran acquiring GX actually helps datascope's positioning. State: PR #6 open. Tool demo-safe. 283 tests pass. Move 4 (Growth) future. Next: Merge PR #6. Then rename folder to "datascope" and begin client outreach. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
MsShawnP
added a commit
that referenced
this pull request
May 20, 2026
70 new tests across test_report_html.py (37) and test_report_annotated_excel.py (33) covering structure, styling, severity coloring, XSS safety, edge cases, and palette integration. Also fixes undefined NAVY reference in pdf.py (rebase artifact from PR #6) — replaced with CHICAGO_20 per Lailara design system. Co-Authored-By: Claude Opus 4.6 <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
Pre-launch readiness pass before sharing demo links with prospective clients. Driven by a full project audit (AUDIT.md Round 2).
--sheetno longer crashes (friendly error with available sheets listed), grammar bugs fixed in narrative templates, help text updated for Parquet, numpy added to dev deps.dev/Test plan
datascope samples/input/sample_mixed_types.xlsx— produces branded PDF with no crashesdatascope samples/input/sample_mixed_types.xlsx --sheet NonExistent— prints friendly error, exit 1datascope samples/input/sample_mixed_types.xlsx --format html— HTML has favicon in browser tabdatascope samples/input/sample_mixed_types.xlsx --format json— JSON has"generator"fielddatascope somefile.csv --sheet X— prints warning that --sheet is ignoredpython -m pytest tests/— 283 tests passruff check datascope/ tests/— clean🤖 Generated with Claude Code