DEL: Remove unused delivery variants

Author: robertmaierle

CHANGELOG.md

@@ -16,6 +16,7 @@
 - Added missing `IndexTs()` method to `v1::InstrumentDefMsg`, `v2::InstrumentDefMsg`,
   and `v1::SymbolMappingMsg`
 - Fixed error in templated overload of `DbnDecoder::EncodeRecord()`
+- Removed unused `Disk` and `S3` variants from `Delivery` enum
 
 ## 0.39.1 - 2025-07-22
 

include/databento/enums.hpp

@@ -30,8 +30,6 @@ enum class SplitDuration : std::uint8_t {
 // Represents how a batch job will be delivered.
 enum class Delivery : std::uint8_t {
   Download,
-  S3,
-  Disk,
 };
 
 // The current state of a batch job.

src/enums.cpp

@@ -62,12 +62,6 @@ const char* ToString(Delivery delivery) {
     case Delivery::Download: {
       return "download";
     }
-    case Delivery::S3: {
-      return "s3";
-    }
-    case Delivery::Disk: {
-      return "disk";
-    }
     default: {
       return "unknown";
     }
@@ -957,12 +951,6 @@ Delivery FromString(const std::string& str) {
   if (str == "download") {
     return Delivery::Download;
   }
-  if (str == "s3") {
-    return Delivery::S3;
-  }
-  if (str == "disk") {
-    return Delivery::Disk;
-  }
   throw InvalidArgumentError{"FromString<Delivery>", "str",
                              "unknown value '" + str + '\''};
 }