Merge pull request #38 from rasnes/EtL-uppercase-all-stocks-to-cli

fix: allow lowercase tickers as cli arguments
rasnes · Jan 3, 2025 · bc19e3e · bc19e3e
2 parents d19d3c5 + 252eff6
commit bc19e3e
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/EtL/cmd/eod_backfill.go b/EtL/cmd/eod_backfill.go
@@ -25,7 +25,7 @@ func newBackfillCmd() *cobra.Command {
 			}
 			defer pipeline.Close()
 
-			tickers := strings.Split(args[0], ",") // Assuming comma-separated tickers
+			tickers := strings.Split(strings.ToUpper(args[0]), ",") // Convert tickers to uppercase
 			nSuccess, err := pipeline.BackfillEndOfDay(tickers)
 			if err != nil {
 				return fmt.Errorf("error backfilling tickers: %w", err)

diff --git a/EtL/cmd/fundamentals.go b/EtL/cmd/fundamentals.go
@@ -48,7 +48,7 @@ func newFundamentalsDailyCmd() *cobra.Command {
 
 			var tickerSlice []string
 			if tickers != "" {
-				tickerSlice = strings.Split(tickers, ",")
+				tickerSlice = strings.Split(strings.ToUpper(tickers), ",")
 			}
 
 			var skipTickerSlice []string
@@ -138,7 +138,7 @@ func newStatementsCmd() *cobra.Command {
 
 			var tickerSlice []string
 			if tickers != "" {
-				tickerSlice = strings.Split(tickers, ",")
+				tickerSlice = strings.Split(strings.ToUpper(tickers), ",")
 			}
 
 			var skipTickerSlice []string