Repository navigation
Migrate Python and Cookiecutter options to native Unified Settings - #8662
Merged
Stella Huang (StellaHuang95) merged 10 commits intoOct 30, 2026
Merged
Stella Huang (StellaHuang95) merged 10 commits into
Stella Huang (StellaHuang95) merged 10 commits into
Conversation
Add native Python and Cookiecutter settings, one-time legacy import, typed persistence, change notifications, navigation, and VS 2026 packaging. Keep hidden preferences and operational state in legacy storage. Add regression coverage and document remaining integration limitations. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
|
Azure Pipelines: There may be pipelines that require an authorized user to comment /azp run to run. |
Use one completion flag and one import batch per component, assume native settings are always enabled, restore legacy setting order, hide Cookiecutter General from navigation, and exclude the non-UI analysis preference. Remove the implementation plan from the PR. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Contributor
|
/azp run |
|
Azure Pipelines: 1 pipeline(s) were filtered out due to trigger conditions. |
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Contributor
|
/azp run |
|
Azure Pipelines: 1 pipeline(s) were filtered out due to trigger conditions. |
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Stella Huang (StellaHuang95)
marked this pull request as ready for review
October 30, 2026 17:16
Contributor
|
/azp run PTVS-Build |
|
Azure Pipelines: Successfully started running 1 pipeline(s). |
Stella Huang (StellaHuang95)
approved these changes
Oct 30, 2026
Stella Huang (StellaHuang95)
enabled auto-merge (squash)
October 30, 2026 17:23
Contributor
|
🔒 Automated review in progress — Stella Huang (@StellaHuang95) is auto-reviewing this PR. |
Bill Schnurr (bschnurr)
approved these changes
Oct 30, 2026
Bill Schnurr (bschnurr)
left a comment
Member
There was a problem hiding this comment.
Approved via Review Center.
Heejae Chang (heejaechang)
approved these changes
Oct 30, 2026
Heejae Chang (heejaechang)
left a comment
There was a problem hiding this comment.
Approved via Review Center.
Rich Chiodo (rchiodo)
approved these changes
Oct 30, 2026
Rich Chiodo (rchiodo)
left a comment
Contributor
There was a problem hiding this comment.
Approved via Review Center.
auto-merge was automatically disabled
October 30, 2026 18:09
Head branch was pushed to by a user without write access
Contributor
|
/azp run PTVS-Build |
|
Azure Pipelines: Successfully started running 1 pipeline(s). |
Stella Huang (StellaHuang95)
approved these changes
Oct 30, 2026
Stella Huang (StellaHuang95)
enabled auto-merge (squash)
October 30, 2026 18:34
Bill Schnurr (bschnurr)
approved these changes
Oct 30, 2026
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
auto-merge was automatically disabled
October 30, 2026 19:07
Head branch was pushed to by a user without write access
Contributor
|
/azp run PTVS-Build |
Stella Huang (StellaHuang95)
enabled auto-merge (squash)
October 30, 2026 19:27
|
Azure Pipelines: Successfully started running 1 pipeline(s). |
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
auto-merge was automatically disabled
October 30, 2026 20:24
Head branch was pushed to by a user without write access
|
Contributor
|
/azp run PTVS-Build |
|
Azure Pipelines: Successfully started running 1 pipeline(s). |
Stella Huang (StellaHuang95)
enabled auto-merge (squash)
October 30, 2026 21:01
Stella Huang (StellaHuang95)
approved these changes
Oct 30, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.



Summary
cookiecutter.general.hideFromNavigationtotrue.black,autopep8,yapf) and the approved reset defaults: Type Checking Off and build-error prompting On. No external settings providers, independent roaming rules, or legacy import/export compatibility were added.General settings ordering
b27f44817is deployed to VS 18 Main using the default root suffix; installed files match the current VSIX by SHA-256.Legacy UI cleanup
Validation
git diff --checkpasses. The pre-existing unrelatedpackage.jsonchanges remain excluded.b27f44817deployed into the explicitly approved VS 18 Main default root suffix. Core installer exited successfully; all 26 installed Core DLL/resource/manifest/registry files match the latest VSIX by SHA-256. Cookiecutter remains current, with its DLL, registry definition, and settings manifest also verified against its VSIX. The six migrated General settings are placed in the inline group below the editor controls. Settings were not reset. This verifies deployment, not installed-VS runtime acceptance.Approved editor-setting exception
TextEditor.Python.Auto List Params. VS promotes the declaredapiOnlymapping to effectiveFullWithoutDefaultsbecause this editor setting is shared/roamed. The user explicitly approved retaining that standard synchronization/roaming after the initial legacy-to-USX migration. This is an approved exception, not an unresolved blocker. The two component import-completed flags remain unchanged; no additional programmatic import passes, external settings provider, or independent roaming rule are introduced.Remaining runtime validation