Skip to content

chore: move bwrap config helpers into dedicated module#15898

Merged
viyatb-oai merged 3 commits intomainfrom
codex/viyatb/bwrap-config-module-followup
Mar 26, 2026
Merged

chore: move bwrap config helpers into dedicated module#15898
viyatb-oai merged 3 commits intomainfrom
codex/viyatb/bwrap-config-module-followup

Conversation

@viyatb-oai
Copy link
Copy Markdown
Collaborator

Summary

  • move the bwrap PATH lookup and warning helpers out of config/mod.rs
  • move the related tests into a dedicated bwrap_tests.rs file

Validation

  • git diff --check
  • skipped heavier local tests per request

Follow-up to #15791.

@viyatb-oai viyatb-oai changed the title Move bwrap config helpers into dedicated module Mar 26, 2026
@viyatb-oai viyatb-oai requested a review from bolinfest March 26, 2026 18:47
Base automatically changed from codex/viyatb/fix-bwrap-system-path to main March 26, 2026 19:13
@viyatb-oai viyatb-oai force-pushed the codex/viyatb/bwrap-config-module-followup branch from d422b7e to 9f63587 Compare March 26, 2026 19:18
@viyatb-oai viyatb-oai marked this pull request as ready for review March 26, 2026 19:27
@viyatb-oai viyatb-oai requested a review from bolinfest March 26, 2026 19:58
@viyatb-oai viyatb-oai merged commit b52abff into main Mar 26, 2026
87 of 92 checks passed
@viyatb-oai viyatb-oai deleted the codex/viyatb/bwrap-config-module-followup branch March 26, 2026 22:16
@github-actions github-actions bot locked and limited conversation to collaborators Mar 26, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

2 participants