clone/codex
1
0
Fork 0
mirror of https://github.com/openai/codex.git synced 2026-05-24 13:04:29 +00:00
Code Issues Packages Projects Releases Wiki Activity

Fix environment manager clippy lints

Browse Source 
Co-authored-by: Codex <noreply@openai.com>
This commit is contained in:
starr-openai
2026-05-01 13:07:58 -07:00
parent 729d8109a3
commit a970e46442
1 changed files with 10 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

16
codex-rs/exec-server/src/environment.rs
View File

@@ -73,12 +73,14 @@ impl EnvironmentManager {
/// Builds a test-only manager with environment access disabled.
pub fn disabled_for_tests(local_runtime_paths: ExecServerRuntimePaths) -> Self {
Self::from_environments(
match Self::from_environments(
HashMap::new(),
local_runtime_paths,
DefaultEnvironmentSelection::Disabled,
)
.expect("disabled test environment manager")
) {
Ok(manager) => manager,
Err(err) => panic!("disabled test environment manager: {err}"),
}
}
/// Builds a test-only manager from a raw exec-server URL value.
@@ -105,12 +107,14 @@ impl EnvironmentManager {
) -> Self {
let provider = DefaultEnvironmentProvider::new(exec_server_url);
let provider_environments = provider.environments(&local_runtime_paths);
Self::from_environments(
match Self::from_environments(
provider_environments,
local_runtime_paths,
provider.default_environment_selection(),
)
.expect("default provider should create valid environments")
) {
Ok(manager) => manager,
Err(err) => panic!("default provider should create valid environments: {err}"),
}
}
/// Builds a manager from a provider-supplied startup snapshot.