{CI} Update the --dependent-file parameter to be an optional argument…

… to avoid introducing a breaking change. (Azure#6521)
wangzelin007 · Jul 17, 2023 · b442cf0 · b442cf0
1 parent 332a2b6
commit b442cf0
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 3 deletions.
diff --git a/scripts/ci/index_ref_doc.py b/scripts/ci/index_ref_doc.py
@@ -55,13 +55,15 @@ def test(self):
                 if dep_url.get(ext_name):
                     dep_file = get_whl_from_url(dep_url[ext_name][0], dep_url[ext_name][1], self.whl_dir)
                 else:
-                    dep_file = ''
+                    dep_file = None
                 ext_file = get_whl_from_url(ext_url, filename, self.whl_dir)
                 ref_doc_out_dir = os.path.join(REF_DOC_OUT_DIR, ext_name)
                 if not os.path.isdir(ref_doc_out_dir):
                     os.mkdir(ref_doc_out_dir)
                 script_args = [sys.executable, REF_GEN_SCRIPT, '--extension-file', ext_file, '--output-dir',
-                               ref_doc_out_dir, '--dependent-file', dep_file]
+                               ref_doc_out_dir]
+                if dep_file:
+                    script_args.extend(['--dependent-file', dep_file])
                 try:
                     check_call(script_args)
                 except CalledProcessError as e:

diff --git a/scripts/refdoc/generate.py b/scripts/refdoc/generate.py
@@ -82,6 +82,6 @@ def _type_path(val):
     parser.add_argument('-o', '--output-dir', dest='output_dir',
                         help='Path to place the generated documentation. By default, a temporary directory will be created.', required=False, type=_type_path)
     parser.add_argument('-d', '--dependent-file', dest='dep_file',
-                        help='Path to the dependent extension .whl file.', required=True, type=str)
+                        help='Path to the dependent extension .whl file.', required=False, type=str)
     args = parser.parse_args()
     generate(args.ext_file, args.output_dir, args.dep_file)