Skip to content

Commit

Permalink
Merge branch 'NotNull'
Browse files Browse the repository at this point in the history
# Conflicts:
#	Signum.Engine/Engine/SchemaSynchronizer.cs
#	Signum.Engine/Patterns/DeletePart.cs
#	Signum.Engine/Schema/Schema.Save.cs
#	Signum.Entities/ValidationAttributes.cs
#	Signum.React/ApiControllers/QueryController.cs
#	Signum.React/Scripts/SelectorModal.tsx
#	Signum.React/Scripts/Signum.Entities.ts
#	Signum.React/Signum.React.csproj
  • Loading branch information
olmobrutall committed Apr 2, 2019
2 parents 8ee55ca + cf7f4e6 commit 2033a7d
Show file tree
Hide file tree
Showing 273 changed files with 5,522 additions and 5,894 deletions.
234 changes: 0 additions & 234 deletions Signum.Analyzer/Signum.Analyzer.Test/AutoPropertyTest.cs

This file was deleted.

Loading

3 comments on commit 2033a7d

@olmobrutall
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nullable Reference Types merged! (C# 8, VS2019)

Today Visual Studio 2019 has been released! Go Install it in https://visualstudio.microsoft.com/! and check what's new in C# 8.0 with this video and docs.

And today, I've merged the NotNull branch into master, and created a legacy masterCS7 for the ones that can not update yet.

This is a big change that will affect a lot of your C# code, all the instructions and motivations for the changes where already written some months ago (28c71a1#comments) and Southwind has been updated accordingly, so every new application will be using nullable reference types from now on.

It's time to update your apps and enjoy a world without NullReferenceException!

@MehdyKarimpour
Copy link
Contributor

@MehdyKarimpour MehdyKarimpour commented on 2033a7d Apr 3, 2019 via email

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@olmobrutall
Copy link
Collaborator Author

@olmobrutall olmobrutall commented on 2033a7d Apr 12, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please sign in to comment.