Skip to content

Commit bb34751

Browse files
amuraruCopilot
andauthored
fix(ci): make dependency updater executable (#73)
Preserve script permissions in regression fixtures and invoke the updater as CI does so missing executable permissions are detected. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
1 parent d8b099f commit bb34751

2 files changed

Lines changed: 2 additions & 2 deletions

File tree

‎scripts/test_update_deps.py‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -85,7 +85,7 @@ def setUp(self):
8585
for name in (*FILES, "update-go-deps.sh"):
8686
target = self.root / name
8787
target.parent.mkdir(parents=True, exist_ok=True)
88-
shutil.copyfile(ROOT / name, target)
88+
shutil.copy2(ROOT / name, target)
8989
self.bin = self.root / "bin"
9090
self.bin.mkdir()
9191
for name in ("go", "gh", "docker", "make", "curl"):
@@ -95,7 +95,7 @@ def setUp(self):
9595

9696
def run_updater(self, **env):
9797
return subprocess.run(
98-
["bash", "update-go-deps.sh"],
98+
["bash", "-e", "-c", "./update-go-deps.sh"],
9999
cwd=self.root,
100100
env={**os.environ, "PATH": f"{self.bin}:{os.environ['PATH']}", **env},
101101
text=True,

‎update-go-deps.sh‎

100644100755
File mode changed.

0 commit comments

Comments
 (0)