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

Avoid declaring RazorPage<T>.Model as nullable by default #39332

Merged
merged 1 commit into from Jan 7, 2022

Conversation

pranavkm
Copy link
Contributor

@pranavkm pranavkm commented Jan 5, 2022

Contributes to #37510

@ghost ghost added the area-mvc Includes: MVC, Actions and Controllers, Localization, CORS, most templates label Jan 5, 2022
@pranavkm pranavkm marked this pull request as ready for review January 6, 2022 01:45
@pranavkm pranavkm added the api-ready-for-review API is ready for formal API review - https://github.com/dotnet/apireviews label Jan 6, 2022
@ghost
Copy link

ghost commented Jan 6, 2022

Thank you for your API proposal. I'm removing the api-ready-for-review label. API Proposals should be submitted for review through Issues based on this template.

@ghost ghost removed the api-ready-for-review API is ready for formal API review - https://github.com/dotnet/apireviews label Jan 6, 2022
@pranavkm pranavkm requested a review from a team January 6, 2022 21:43
@pranavkm pranavkm merged commit 57f283c into dotnet:main Jan 7, 2022
@pranavkm pranavkm deleted the prkrishn/non-nullable-model branch January 7, 2022 00:49
@ghost ghost added this to the 7.0-preview1 milestone Jan 7, 2022
pranavkm added a commit to pranavkm/aspnetcore that referenced this pull request Jan 10, 2022
Elanis added a commit to Dysnomia-Studio/dysnomia-website that referenced this pull request Jul 14, 2023
Bumps [Microsoft.AspNetCore.TestHost](https://github.com/dotnet/aspnetcore) from 6.0.1 to 6.0.2.
<details>
<summary>Release notes</summary>

*Sourced from [Microsoft.AspNetCore.TestHost's releases](https://github.com/dotnet/aspnetcore/releases).*

> ## .NET 6.0.2
> [Release](https://github.com/dotnet/core/releases/tag/v6.0.2)
</details>
<details>
<summary>Commits</summary>

- [`1dcf7ac`](dotnet/aspnetcore@1dcf7ac) [internal/release/6.0] Update dependencies from dnceng/internal/dotnet-efcore...
- [`608229f`](dotnet/aspnetcore@608229f) [internal/release/6.0] Update dependencies from dnceng/internal/dotnet-runtim...
- [`eb1c0a4`](dotnet/aspnetcore@eb1c0a4) Merge in 'release/6.0' changes
- [`ef499fe`](dotnet/aspnetcore@ef499fe) Revert "[release/6.0] Switch to Windows.Amd64.Server2022.Open ([#39364](dotnet/aspnetcore#39364))" ([#39476](dotnet/aspnetcore#39476))
- [`5e8131b`](dotnet/aspnetcore@5e8131b) Merged PR 20577: Fix build errors
- [`04a113d`](dotnet/aspnetcore@04a113d) Merge in 'release/6.0' changes
- [`2bd9de6`](dotnet/aspnetcore@2bd9de6) Avoid declaring RazorPage\<T>.Model as nullable by default ([#39332](dotnet/aspnetcore#39332)) ([#39416](dotnet/aspnetcore#39416))
- [`6ffe8df`](dotnet/aspnetcore@6ffe8df) Merge in 'release/6.0' changes
- [`d9521ac`](dotnet/aspnetcore@d9521ac) Don't exclude System.IO.Pipelines from TargetingPack version check ([#39302](dotnet/aspnetcore#39302))
- [`3b49ab1`](dotnet/aspnetcore@3b49ab1) Merge in 'release/6.0' changes
- Additional commits viewable in [compare view](dotnet/aspnetcore@v6.0.1...v6.0.2)
</details>

<br />

Co-authored-by: Elanis <elanis@hotmail.com>
Reviewed-on: https://gitea.dysnomia.studio/elanis/dysnomia-website/pulls/26
Co-authored-by: elanis <elanis@noreply.example.org>
Co-committed-by: elanis <elanis@noreply.example.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-mvc Includes: MVC, Actions and Controllers, Localization, CORS, most templates
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants