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

Use the source-built version of ref packs and don't use app host when building in source-build #64055

Merged
merged 2 commits into from
Oct 10, 2022

Conversation

crummel
Copy link
Contributor

@crummel crummel commented Sep 15, 2022

No description provided.

@crummel
Copy link
Contributor Author

crummel commented Sep 28, 2022

@jaredpar This is the fix we talked about earlier, don't know if you're the right person to review it? Thanks!

@jaredpar jaredpar requested a review from 333fred October 6, 2022 23:20
@jaredpar jaredpar added this to the 17.5 milestone Oct 6, 2022
@jaredpar
Copy link
Member

jaredpar commented Oct 6, 2022

@333fred PTAL

@333fred
Copy link
Member

333fred commented Oct 10, 2022

/azp run roslyn-CI

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@333fred 333fred merged commit 17b095e into dotnet:main Oct 10, 2022
@ghost ghost modified the milestones: 17.5, Next Oct 10, 2022
@crummel
Copy link
Contributor Author

crummel commented Oct 10, 2022

Thanks!

333fred added a commit to 333fred/roslyn that referenced this pull request Oct 10, 2022
* upstream/main: (252 commits)
  Use the source-built version of ref packs and don't use app host when building in source-build (dotnet#64055)
  Enable rich LSIF hover information. (dotnet#64580)
  Add ChecksumAlgorithm to project snapshot (dotnet#62840)
  Utility for uploading artifact on test failure (dotnet#64578)
  Enable diagnostics
  Revert "Remove unused TS brace completion code"
  Publish additional packages to vssdk feed (dotnet#64571)
  spelling
  Move check
  Simplify SymbolKey implementation
  lint
  Update publish data to test PR validation fix (dotnet#64559)
  Simplify
  Update src/Workspaces/SharedUtilitiesAndExtensions/Compiler/Core/SymbolKey/SymbolKey.cs
  Lint
  Proper equality checks
  Update src/Workspaces/Remote/Core/RemoteCallback.cs
  Revert "Not wait for solution crawler because it can be very busy"
  Add and use TargetFramework.Net70 (dotnet#64490)
  Not wait for solution crawler because it can be very busy
  ...
crummel added a commit to crummel/dotnet_roslyn that referenced this pull request Oct 11, 2022
… building in source-build (dotnet#64055)

* Use the source-built version of ref packs and don't use app host when building in source-build.

* Fix typo.
jaredpar pushed a commit that referenced this pull request Oct 12, 2022
… building in source-build (#64055) (#64650)

* Use the source-built version of ref packs and don't use app host when building in source-build.

* Fix typo.
@RikkiGibson RikkiGibson modified the milestones: Next, 17.5 P1 Oct 24, 2022
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.

4 participants