feat!: include merge functionality as a flag for import instead of …

…a separate command (#73) BREAKING CHANGE: removed the `merge` command and integrated it into `import` via `--enable-merging=true`
miracum · Nov 20, 2024 · b856a1d · b856a1d
1 parent 9f6b11f
commit b856a1d
Show file tree

Hide file tree

Showing 3 changed files with 405 additions and 575 deletions.
diff --git a/README.md b/README.md
@@ -42,7 +42,8 @@ dotnet run --project src/PathlingS3Import/ -- import \
     --s3-object-name-prefix=staging/ \
     --enable-metrics=true \
     --pushgateway-endpoint=http://localhost:9091/ \
-    --dry-run=false
+    --dry-run=true \
+    --enable-merging=true
 ```
 
 Or to test importing from a checkpoint:
@@ -59,19 +60,6 @@ dotnet run --project src/PathlingS3Import/ -- import \
     --dry-run=false
 ```
 
-### merge
-
-```sh
-dotnet run --project src/PathlingS3Import/ -- merge \
-    --s3-endpoint=http://localhost:9000 \
-    --s3-access-key=admin \
-    --s3-secret-key=miniopass \
-    --s3-bucket-name=fhir \
-    --s3-object-name-prefix=staging/ \
-    --max-merged-bundle-size=10 \
-    --dry-run=true
-```
-
 ### Run E2E Tests
 
 ```sh