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

Add reference to ExternalAccess.RazorCompiler #8142

Closed
wants to merge 4 commits into from

Conversation

jjonescz
Copy link
Member

@jjonescz jjonescz commented Jan 17, 2023

We had multiple sets of roslyn references, differing between the compiler and the tooling. This consolidates those versions and updates the compiler references to be inline with the tooling.
@jjonescz jjonescz force-pushed the ea-razor-compiler branch 5 times, most recently from 4063040 to 69cfb34 Compare January 19, 2023 18:11
@ryanbrandenburg
Copy link
Contributor

@jjonescz be sure to run this against https://devdiv.visualstudio.com/DevDiv/_build?definitionId=15591&_a=summary as well since we need to make sure that the changes work in both VSPreview and VSMain. In order for that pipeline to work for PR builds you need to manually push your branch to both the base GitHub repo (not your fork) and to razor's repo in dnceng (I know, it's super annoying). Feel free to message me if you run into trouble.

@jjonescz jjonescz force-pushed the ea-razor-compiler branch 2 times, most recently from 29e93ac to c9d22d8 Compare February 3, 2023 10:03
@jjonescz jjonescz marked this pull request as ready for review February 3, 2023 11:50
@jjonescz jjonescz requested review from a team as code owners February 3, 2023 11:50
@jjonescz jjonescz marked this pull request as draft February 3, 2023 17:27
@jjonescz jjonescz mentioned this pull request Feb 8, 2023
3 tasks
@jjonescz jjonescz marked this pull request as ready for review February 8, 2023 10:54
NuGet.config Outdated Show resolved Hide resolved
@jjonescz
Copy link
Member Author

Included in #8242.

@jjonescz jjonescz closed this Feb 16, 2023
@jjonescz jjonescz deleted the ea-razor-compiler branch February 16, 2023 08: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.

3 participants