Skip to content

Productize PEP 723 inline script preview - #33

Closed
StellaHuang95 wants to merge 1 commit into
mainfrom
stellahuang95-pep723-analysis
Closed

StellaHuang95 wants to merge 1 commit into
mainfrom
stellahuang95-pep723-analysis

Conversation

@StellaHuang95

Copy link
Copy Markdown
Owner

Summary

  • add the public python-envs.inlineScripts.enabled preview setting with a default of false, requiresReload, and onExP rollout support
  • use one activation-latched decision across inline-script commands, diagnostics, manager registration, CodeLens, and quick fixes
  • document the preview and make automated test hosts explicitly default the feature off
  • add manifest and behavior coverage for the rollout contract

Validation

  • npm run lint
  • npm run compile-tests
  • npm run compile
  • npm run unittest — 3,295 passing, 9 pending
  • smoke tests — 31 passing, 1 pending
  • E2E tests — 4 passing
  • manual VS Code verification of settings precedence, command visibility, CodeLens/interpreter routing, and both true→false and false→true reload latching

The full integration run had 60 passing and 8 pending tests plus one pre-existing timeout in Created environment appears in discovery. The committed fixture selects the interactive system installer; the targeted test passes with a venv-configured fixture.

Add a default-off experimental setting and use one activation-latched decision across commands, diagnostics, managers, CodeLens, and quick fixes. Update documentation and test coverage for gradual rollout.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Copilot-Session: 11d44bb8-39ba-4f1f-9a2c-379198878a3a
@StellaHuang95
StellaHuang95 force-pushed the stellahuang95-pep723-analysis branch from c9026cb to 7877b8c Compare October 20, 2026 18:45
@StellaHuang95

Copy link
Copy Markdown
Owner Author

Superseded by microsoft#1938.

@StellaHuang95
StellaHuang95 deleted the stellahuang95-pep723-analysis branch October 23, 2026 18:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant