Merge branch 'main' into Issue-79-sql-script

Author: zacharysyoung

.github/workflows/ci.yaml

@@ -0,0 +1,28 @@
+name: CI
+
+on:
+  push:
+    branches: [ main ]
+  pull_request:
+    branches: [ main ]
+
+jobs:
+  test:
+    runs-on: ubuntu-latest
+    steps:
+      - name: Checkout code
+        uses: actions/checkout@v3
+
+      - name: Set up Go
+        uses: actions/setup-go@v5
+        with:
+          go-version: stable
+
+      - name: Install dependencies
+        run: go get .
+
+      - name: Formatting
+        run: if [ "$(gofmt -s -l . | wc -l)" -gt 0 ]; then gofmt -s -l .;  exit 1; fi
+
+      - name: Run tests
+        run: make test

Makefile

@@ -1,7 +1,5 @@
 BIN_DIR=bin
 DIST_DIR=dist
-CMD_DIR=cmd
-CSV_DIR=csv
 EXECUTABLE=gocsv
 
 .DEFAULT_GOAL := bin
@@ -11,8 +9,7 @@ dist:
 	bash scripts/build-dist.sh
 
 test:
-	cd $(CMD_DIR) && go test -cover
-	cd $(CSV_DIR) && go test -cover
+	go test -cover ./...
 
 bin:
 	bash scripts/build-bin.sh

README.md

@@ -369,7 +369,7 @@ Arguments:
 - `--case-insensitive` (optional, shorthand `-i`) Use this flag to specify a case insensitive match for replacement rather than the default case sensitive match.
 - `--repl` String to use for replacement.
 
-Note that if you have a capture group in the `--regex` argument you can reference that in the replacement argument using `"\$1"` for the first capture group, `"\$2"` for the second capture group, etc.
+Note that if you have a capture group in the `--regex` argument you can reference that in the replacement argument using `"\$1"` for the first capture group, `"\$2"` for the second capture group, etc. In Windows cmd, the `\` escape is unnecessary, and the capture group references should be specifed as `"$1"` and `"$2"`.
 
 ### sample
 
@@ -428,13 +428,14 @@ Split a CSV into multiple files.
 Usage:
 
 ```shell
-gocsv split --max-rows N [--filename-base FILENAME] FILE
+gocsv split --max-rows N [--filename-base FILENAME] [--width N] FILE
 ```
 
 Arguments:
 
 - `--max-rows` Maximum number of rows per final CSV.
 - `--filename-base` (optional) Prefix of the resulting files. The file outputs will be appended with `"-1.csv"`,`"-2.csv"`, etc. If not specified, the base filename will be the same as the base of the input filename, unless the input is specified by standard input. If so, then the base filename will be `out`.
+- `--width` (optional) Minimum width of the numeric suffix, zero-padded if necessary. For example, `--width 3` results in filenames like `"file-001.csv"`, `"file-002.csv"`, etc.
 
 ### sql
 
@@ -568,15 +569,15 @@ The command defaults to writing all converted sheets to a directory with the sam
 
 Usage:
 
-```none
+```shell
 gocsv xlsx [--list-sheets | --dirname DIRNAME | --sheet SHEET] FILE
 ```
 
 Arguments:
 
 - `--list-sheets` (optional) List the sheets, by index and name, in the XLSX file.
 - `--sheet` (optional) Specify a single sheet, by index or name, to convert and write to stdout.
-- `--dirname` (optional) Specify the name of the directory for the converted sheets.  The command defaults to the same name as `FILE`, minus the extension.
+- `--dirname` (optional) Specify the name of the directory for the converted sheets. The command defaults to the same name as `FILE`, minus the extension.
 
 Only one option can be used; multiple options cannot be combined.
 

cmd/delimiter.go

@@ -20,10 +20,10 @@ func (sub *DelimiterSubcommand) Description() string {
 	return "Change the delimiter being used for a CSV."
 }
 func (sub *DelimiterSubcommand) SetFlags(fs *flag.FlagSet) {
-	fs.StringVar(&sub.inputDelimiter, "input", "", "Input delimiter")
-	fs.StringVar(&sub.inputDelimiter, "i", "", "Input delimiter (shorthand)")
-	fs.StringVar(&sub.outputDelimiter, "output", "", "Output delimiter")
-	fs.StringVar(&sub.outputDelimiter, "o", "", "Output delimiter (shorthand)")
+	fs.StringVar(&sub.inputDelimiter, "input", ",", "Input delimiter")
+	fs.StringVar(&sub.inputDelimiter, "i", ",", "Input delimiter (shorthand)")
+	fs.StringVar(&sub.outputDelimiter, "output", ",", "Output delimiter")
+	fs.StringVar(&sub.outputDelimiter, "o", ",", "Output delimiter (shorthand)")
 }
 
 func (sub *DelimiterSubcommand) Run(args []string) {