Skip to content

Remove ISourceFile Uri#20037

Merged
shenglol merged 1 commit into
mainfrom
shenglol/source-file-uri-cleanup
Jul 10, 2026
Merged

Remove ISourceFile Uri#20037
shenglol merged 1 commit into
mainfrom
shenglol/source-file-uri-cleanup

Conversation

@shenglol

@shenglol shenglol commented Jul 10, 2026

Copy link
Copy Markdown
Contributor

Removes the obsolete ISourceFile.Uri property and migrates remaining test helpers/call sites to use FileHandle.Uri / IOUri instead.

Summary:

  • Remove Uri from the ISourceFile contract and concrete source file classes.
  • Update test template helpers to accept IOUri.
  • Move remaining test usage from sourceFile.Uri to sourceFile.FileHandle.Uri.
  • Update the extension config inheritance test to use InMemoryTestFileSet.

Validation:

  • dotnet build .\Bicep.sln /property:GenerateFullPaths=true /consoleloggerparameters:NoSummary
Microsoft Reviewers: Open in CodeFlow

@github-actions

github-actions Bot commented Jul 10, 2026

Copy link
Copy Markdown
Contributor

Test this change out locally with the following install scripts (Action run 29124273362)

VSCode
  • Mac/Linux
    bash <(curl -Ls https://aka.ms/bicep/nightly-vsix.sh) --run-id 29124273362
  • Windows
    iex "& { $(irm https://aka.ms/bicep/nightly-vsix.ps1) } -RunId 29124273362"
Azure CLI
  • Mac/Linux
    bash <(curl -Ls https://aka.ms/bicep/nightly-cli.sh) --run-id 29124273362
  • Windows
    iex "& { $(irm https://aka.ms/bicep/nightly-cli.ps1) } -RunId 29124273362"

@shenglol shenglol merged commit 75f62c3 into main Jul 10, 2026
43 of 44 checks passed
@shenglol shenglol deleted the shenglol/source-file-uri-cleanup branch July 10, 2026 21:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants