Skip to content

Commit 6c1ca2c

Browse files
authored
Merge pull request #72 from cadenmyers13/print-info-ci
CI: handle subprocess for windows
2 parents 79170b1 + 80241d2 commit 6c1ca2c

File tree

2 files changed

+31
-11
lines changed

2 files changed

+31
-11
lines changed

news/print-info-ci.rst

Lines changed: 23 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,23 @@
1+
**Added:**
2+
3+
* No news needed.
4+
5+
**Changed:**
6+
7+
* <news item>
8+
9+
**Deprecated:**
10+
11+
* <news item>
12+
13+
**Removed:**
14+
15+
* <news item>
16+
17+
**Fixed:**
18+
19+
* <news item>
20+
21+
**Security:**
22+
23+
* <news item>

tests/test_packsmanager.py

Lines changed: 8 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -372,27 +372,24 @@ def test_print_info(packs_to_install, expected, example_cases, capsys):
372372
case5dir = example_cases / "case5"
373373
env_dir = case5dir / "fake_env"
374374
req_dir = case5dir / "requirements" / "packs"
375+
# Handle Windows path format
376+
env_dir_str = env_dir.as_posix()
377+
shell = os.name == "nt"
375378
subprocess.run(
376-
["conda", "create", "-y", "-p", str(env_dir)],
379+
["conda", "create", "-y", "-p", env_dir_str],
377380
check=True,
378381
capture_output=True,
379382
text=True,
383+
shell=shell,
380384
)
381385
for pack in packs_to_install:
382-
req_file = req_dir / f"{pack}.txt"
386+
req_file = (req_dir / f"{pack}.txt").as_posix()
383387
subprocess.run(
384-
[
385-
"conda",
386-
"install",
387-
"-y",
388-
"--file",
389-
str(req_file),
390-
"-p",
391-
str(env_dir),
392-
],
388+
["conda", "install", "-y", "--file", req_file, "-p", env_dir_str],
393389
check=True,
394390
capture_output=True,
395391
text=True,
392+
shell=shell,
396393
)
397394
pm = PacksManager(root_path=case5dir)
398395
pm.print_info()

0 commit comments

Comments
 (0)