Merge pull request #429 from lsst/tickets/DM-35917

DM-35917: Note that --config no longer splits on comma
lsst · Aug 26, 2022 · d46986b · d46986b
2 parents 9e4e920 + 40f2735
commit d46986b
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/doc/changes/DM-35917.api.rst b/doc/changes/DM-35917.api.rst
@@ -0,0 +1,3 @@
+The ``ingest-raws`` and ``define-visits`` subcommands no longer allow multiple config settings within a single ``--config`` option.
+We have decided that it is too dangerous to split on comma in the general case and so have removed that facility to be consistent with other commands.
+Use multiple ``--config`` options instead.
diff --git a/tests/test_cliCmdTestIngest.py b/tests/test_cliCmdTestIngest.py
@@ -76,7 +76,9 @@ def test_configMulti(self):
                 "-c",
                 "foo=1",
                 "--config",
-                "bar=2,baz=3",
+                "bar=2",
+                "--config",
+                "baz=3",
             ],
             self.makeExpected(
                 repo="repo",