Skip to content

docs: add release badge and real-world recipes#18

Merged
vmvarela merged 1 commit intomasterfrom
fix/issue-6-readme
Mar 2, 2026
Merged

docs: add release badge and real-world recipes#18
vmvarela merged 1 commit intomasterfrom
fix/issue-6-readme

Conversation

@vmvarela
Copy link
Owner

@vmvarela vmvarela commented Mar 2, 2026

Closes #6

What changed

  • Added latest release badge (the only badge missing from the acceptance criteria).
  • Added a Recipes section with 6 concrete, copy-pasteable examples covering the patterns requested in the issue:
    • Top-N by column
    • Deduplication
    • Missing value detection
    • Date range filter
    • Derived/computed columns
    • Conditional pivot-style aggregation
  • Existing 5 usage examples in the Usage section are retained unchanged.

Acceptance criteria check

Criterion Status
Installation section (binary + build from source) ✅ already present
Basic usage example ✅ already present
≥ 5 real-world examples ✅ 5 in Usage + 6 in Recipes
Input format documented (header row, table t) ✅ already present
Exit codes and error messages documented ✅ already present
Build status badge ✅ already present
Latest release badge added
License badge ✅ already present
--help flag documented ✅ already in Flags table

Copilot AI review requested due to automatic review settings March 2, 2026 15:06
@github-actions github-actions bot added the type:docs Documentation only label Mar 2, 2026
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the project README to better meet Issue #6’s documentation acceptance criteria by adding missing repository metadata and expanding practical usage guidance.

Changes:

  • Add a “Latest release” badge alongside existing CI and license badges.
  • Add a new “Recipes” section with 6 copy/paste SQL examples for common CSV workflows.
  • Fix the Exit Codes table separator formatting.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

**Find rows with missing values:**

```sh
$ cat users.csv | sql-pipe 'SELECT * FROM t WHERE email = "" OR email IS NULL'
Copy link

Copilot AI Mar 2, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In these recipes, empty CSV fields are inserted as SQL NULL (see insertRowTyped binding behavior), so email = "" will never match. Consider simplifying to email IS NULL (and optionally OR TRIM(email) = '' if you want to treat whitespace-only values as missing).

Suggested change
$ cat users.csv | sql-pipe 'SELECT * FROM t WHERE email = "" OR email IS NULL'
$ cat users.csv | sql-pipe 'SELECT * FROM t WHERE email IS NULL OR TRIM(email) = ""'

Copilot uses AI. Check for mistakes.
@vmvarela vmvarela merged commit 89f2e4f into master Mar 2, 2026
10 checks passed
@vmvarela vmvarela deleted the fix/issue-6-readme branch March 2, 2026 15:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type:docs Documentation only

Projects

None yet

Development

Successfully merging this pull request may close these issues.

README: installation guide, usage examples, and real-world recipes

2 participants