refactor(env): rename env_resolve.py -> env.py; env_resolve() -> resolve_env_into()
test / run tests/run_tests.py (pull_request) Successful in 13s
test / run tests/run_tests.py (pull_request) Successful in 13s
Module name aligns with the others (manifest, pipelock, network, log) — nouns/noun-y, not verb phrases. The function name now reads naturally at the call site: resolve_env_into(manifest, agent, env_file, args_file).
This commit is contained in:
@@ -19,7 +19,7 @@ from pathlib import Path
|
|||||||
from typing import Iterator, Sequence
|
from typing import Iterator, Sequence
|
||||||
|
|
||||||
from ... import pipelock
|
from ... import pipelock
|
||||||
from ...env_resolve import env_resolve
|
from ...env import resolve_env_into
|
||||||
from ...log import die, info
|
from ...log import die, info
|
||||||
from ...manifest import SshEntry
|
from ...manifest import SshEntry
|
||||||
from ...util import expand_tilde
|
from ...util import expand_tilde
|
||||||
@@ -108,7 +108,7 @@ class DockerBottleBackend(BottleBackend):
|
|||||||
prompt_file.chmod(0o600)
|
prompt_file.chmod(0o600)
|
||||||
|
|
||||||
proxy_plan = self.prepare_proxy(spec, stage_dir)
|
proxy_plan = self.prepare_proxy(spec, stage_dir)
|
||||||
env_resolve(manifest, spec.agent_name, env_file, args_file)
|
resolve_env_into(manifest, spec.agent_name, env_file, args_file)
|
||||||
prompt_file.write_text(agent.prompt)
|
prompt_file.write_text(agent.prompt)
|
||||||
|
|
||||||
allowlist_summary = pipelock.pipelock_allowlist_summary(bottle)
|
allowlist_summary = pipelock.pipelock_allowlist_summary(bottle)
|
||||||
|
|||||||
@@ -97,7 +97,7 @@ def _read_secret_silent(name: str, prompt_body: str) -> str:
|
|||||||
return value
|
return value
|
||||||
|
|
||||||
|
|
||||||
def env_resolve(
|
def resolve_env_into(
|
||||||
manifest: Manifest,
|
manifest: Manifest,
|
||||||
agent: str,
|
agent: str,
|
||||||
env_file: Path,
|
env_file: Path,
|
||||||
Reference in New Issue
Block a user