From 87d7ea0f605b9da00731cb2ee7a5966a6e007c13 Mon Sep 17 00:00:00 2001
From: Michael Terry <michael.terry@childrens.harvard.edu>
Date: Mon, 28 Oct 2024 08:54:51 -0400
Subject: [PATCH] build: bump version to 2.0

And remove deprecated chart-review alias for upload-notes, since this
is a good time to break CLI promises.
---
 compose.yaml            | 10 ----------
 cumulus_etl/__init__.py |  2 +-
 cumulus_etl/cli.py      |  6 +-----
 3 files changed, 2 insertions(+), 16 deletions(-)

diff --git a/compose.yaml b/compose.yaml
index 1cf9853..353ede8 100644
--- a/compose.yaml
+++ b/compose.yaml
@@ -80,10 +80,6 @@ services:
   ctakes-covid:
     extends: ctakes-covid-base
     profiles:
-      # chart-review is a deprecated alias for upload-notes since Jan 2024.
-      # Remove when you feel like it.
-      - chart-review
-      - chart-review-gpu
       - covid-symptom
       - covid-symptom-gpu
       - upload-notes
@@ -93,9 +89,6 @@ services:
     extends: common-base
     image: smartonfhir/cnlp-transformers:negation-0.6.1-cpu
     profiles:
-      # chart-review is a deprecated alias for upload-notes since Jan 2024.
-      # Remove when you feel like it.
-      - chart-review
       - covid-symptom
       - upload-notes
     networks:
@@ -105,9 +98,6 @@ services:
     extends: common-base
     image: smartonfhir/cnlp-transformers:negation-0.6.1-gpu
     profiles:
-      # chart-review-gpu is a deprecated alias for upload-notes-gpu since Jan 2024.
-      # Remove when you feel like it.
-      - chart-review-gpu
       - covid-symptom-gpu
       - upload-notes-gpu
     networks:
diff --git a/cumulus_etl/__init__.py b/cumulus_etl/__init__.py
index c45782d..edeb3f4 100644
--- a/cumulus_etl/__init__.py
+++ b/cumulus_etl/__init__.py
@@ -1,3 +1,3 @@
 """Turns FHIR data into de-identified & aggregated records"""
 
-__version__ = "1.5.0"
+__version__ = "2.0.0"
diff --git a/cumulus_etl/cli.py b/cumulus_etl/cli.py
index 02c2395..acc93ae 100644
--- a/cumulus_etl/cli.py
+++ b/cumulus_etl/cli.py
@@ -16,10 +16,6 @@
 class Command(enum.Enum):
     """Subcommand strings"""
 
-    # chart-review is a deprecated alias of upload-notes since Jan 2024.
-    # Keep as long as you like.
-    # It's a low-usage feature, but it's not a maintenance burden to keep this around.
-    CHART_REVIEW = "chart-review"
     CONVERT = "convert"
     ETL = "etl"
     EXPORT = "export"
@@ -65,7 +61,7 @@ async def main(argv: list[str]) -> None:
         prog += f" {subcommand}"  # to make --help look nicer
     parser = argparse.ArgumentParser(prog=prog)
 
-    if subcommand in {Command.CHART_REVIEW.value, Command.UPLOAD_NOTES.value}:
+    if subcommand == Command.UPLOAD_NOTES.value:
         run_method = upload_notes.run_upload_notes
     elif subcommand == Command.CONVERT.value:
         run_method = convert.run_convert