Skip to content

Refactor: is_match add workspace_root parameter#467

Open
edvilme wants to merge 1 commit intomainfrom
phase4/is-match-workspace-root
Open

Refactor: is_match add workspace_root parameter#467
edvilme wants to merge 1 commit intomainfrom
phase4/is-match-workspace-root

Conversation

@edvilme
Copy link
Copy Markdown
Contributor

@edvilme edvilme commented Apr 7, 2026

Normalization change for shared package extraction.

Tracking: #465 | microsoft/vscode-python-tools-extension-template#290

  • 4.2e Add optional workspace_root param to is_match()

Backwards-compatible. Default None preserves existing behavior.

@edvilme edvilme added the debt label Apr 7, 2026
@edvilme edvilme enabled auto-merge (squash) April 7, 2026 21:02
@edvilme edvilme force-pushed the phase4/is-match-workspace-root branch from b3f0f6b to 1a59be3 Compare April 7, 2026 21:10
rchiodo
rchiodo previously approved these changes Apr 7, 2026
Copy link
Copy Markdown
Contributor

@rchiodo rchiodo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved via Review Center.

@edvilme edvilme force-pushed the phase4/is-match-workspace-root branch from 1a59be3 to 7b8a28c Compare April 7, 2026 21:23
Part of Phase 4 normalization (#465).
Adds optional workspace_root=None parameter for signature alignment with mypy.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@edvilme edvilme requested review from karthiknadig and rchiodo and removed request for karthiknadig April 7, 2026 21:23
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.

2 participants