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

Bump SDK and disable RSG for BasicTestApp #32581

Merged
merged 5 commits into from
May 14, 2021
Merged

Conversation

captainsafia
Copy link
Member

@captainsafia captainsafia commented May 11, 2021

Fixes #31023 and maybe #32237

While we wait for the new incremental source generator API, let's disable the RSG for the BasicTestApp to avoid issues when developing locally.

@Pilchie Pilchie added the area-blazor Includes: Blazor, Razor Components label May 11, 2021
@captainsafia
Copy link
Member Author

Blocked on dotnet/sdk#17491.

@captainsafia captainsafia marked this pull request as ready for review May 13, 2021 22:32
@captainsafia captainsafia requested review from a team as code owners May 13, 2021 22:32
@captainsafia
Copy link
Member Author

This is blocked on dotnet/sdk#17461 now.

@captainsafia captainsafia enabled auto-merge (squash) May 14, 2021 17:45
@captainsafia
Copy link
Member Author

To unblock this PR (and resolve the build flakiness in our repo), I'm skipping the openapi tests for now. We can revisit unskipping them at the next SDK bump if the workload stuff is resolved. There's not much churn in that area anyways so the risk should be minimal compared to the benefit of more reliable builds.

@captainsafia captainsafia merged commit 7551540 into main May 14, 2021
@captainsafia captainsafia deleted the safia/update-sdk-2 branch May 14, 2021 19:19
@ghost ghost added this to the 6.0-preview5 milestone May 14, 2021
@wtgodbe wtgodbe mentioned this pull request May 17, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-blazor Includes: Blazor, Razor Components
Projects
None yet
3 participants