Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Don't compile external module sources #12765

Merged

Conversation

StephenWeatherford
Copy link
Contributor

@StephenWeatherford StephenWeatherford commented Dec 17, 2023

Fixes #12758

Microsoft Reviewers: Open in CodeFlow

Copy link
Contributor

github-actions bot commented Dec 17, 2023

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

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

Copy link
Contributor

github-actions bot commented Dec 17, 2023

Test Results

       66 files   -        33         66 suites   - 33   28m 13s ⏱️ - 26m 47s
10 436 tests  -        15  10 436 ✔️  -        15  0 💤 ±0  0 ±0 
25 367 runs   - 12 682  25 367 ✔️  - 12 682  0 💤 ±0  0 ±0 

Results for commit 19325f6. ± Comparison against base commit 331512e.

♻️ This comment has been updated with latest results.

@StephenWeatherford StephenWeatherford force-pushed the sw/view-sources-header.with-codelens.disable-stuff2 branch from a4b51a7 to af915ad Compare December 18, 2023 17:22
@StephenWeatherford StephenWeatherford force-pushed the sw/view-sources-header.with-codelens.disable-stuff2 branch from af915ad to 19325f6 Compare December 18, 2023 17:26
@StephenWeatherford StephenWeatherford merged commit 0ac075e into main Dec 18, 2023
44 checks passed
@StephenWeatherford StephenWeatherford deleted the sw/view-sources-header.with-codelens.disable-stuff2 branch December 18, 2023 18:07
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.

F12 source: Disable errors, links, hovers, symbol links, etc. when displaying module sources
3 participants