Skip to content

Rename PYWRAP_NO_INSTALL to PYWRAP_INSTALL_DEPS (default 0)#6

Merged
jianliang00 merged 1 commit into
mainfrom
codex/rename-pywrap_no_install-to-pywrap_install_deps
Jan 7, 2026
Merged

Rename PYWRAP_NO_INSTALL to PYWRAP_INSTALL_DEPS (default 0)#6
jianliang00 merged 1 commit into
mainfrom
codex/rename-pywrap_no_install-to-pywrap_install_deps

Conversation

@jianliang00

Copy link
Copy Markdown
Owner

Motivation

  • Replace the old PYWRAP_NO_INSTALL flag with PYWRAP_INSTALL_DEPS to make the intent explicit (enable installs vs disable installs).
  • Make dependency installation opt-in by default so dependencies are only installed when PYWRAP_INSTALL_DEPS=1.
  • Ensure the CLI behavior and logging reflect the new flag semantics.
  • Update documentation to describe the new environment variable and its default.

Description

  • In python3 replaced no_install = _bool_env("PYWRAP_NO_INSTALL", False) with install_deps = _bool_env("PYWRAP_INSTALL_DEPS", False).
  • Inverted the install/skipping logic so dependencies are installed only when install_deps is true and otherwise a log message indicates skipping.
  • Updated README.md to document PYWRAP_INSTALL_DEPS and state the default is 0.
  • Changes touch python3 and README.md.

Testing

  • No automated tests were run on this change.

Codex Task

@jianliang00 jianliang00 merged commit d675ef6 into main Jan 7, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant