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

Efcache 1.3.2 upgrade with netstandard2.1 #44

Merged
merged 2 commits into from
Nov 9, 2020
Merged

Efcache 1.3.2 upgrade with netstandard2.1 #44

merged 2 commits into from
Nov 9, 2020

Conversation

sergevm
Copy link
Contributor

@sergevm sergevm commented Nov 2, 2020

See #43

The serialization of EFCache's ColumnMetadata is broken since dotnet core 2, because System.RuntimeType is no longer marked as serializable. EFCache v1.3.2 adds an ISerializable implementation to the ColumnMetadata struct to handle this. That is the main purpose of this PR.

To handle the compatibility warnings in a dotnet core project, I also propose to add a netstandard2.1 target framework.

Van Meerbeeck, Serge added 2 commits October 22, 2020 08:48
…zation on System.RuntimeType (see moozzyk/EFCache#51) since dotnet core 2.

Added target framework netstandard2.1.
@silentbobbert silentbobbert changed the base branch from master to develop November 8, 2020 09:20
@silentbobbert silentbobbert changed the base branch from develop to feature/pr_44 November 9, 2020 08:47
@silentbobbert silentbobbert merged commit 0c76aaf into silentbobbert:feature/pr_44 Nov 9, 2020
silentbobbert added a commit that referenced this pull request Nov 9, 2020
* Efcache 1.3.2 upgrade with netstandard2.1 (#44)

* Added netstandard2.0 target framework.

* Upgrade to EFCache v1.3.2, because of broken binary formatter serialization on System.RuntimeType (see moozzyk/EFCache#51) since dotnet core 2.

Added target framework netstandard2.1.

Co-authored-by: Van Meerbeeck, Serge <[email protected]>

* Prepare PR for release

Co-authored-by: Ian Robertson <[email protected]>
Co-authored-by: sergevm <[email protected]>
Co-authored-by: Van Meerbeeck, Serge <[email protected]>
Co-authored-by: Ian Robertson <[email protected]>
@silentbobbert
Copy link
Owner

@sergevm - your PR has been merged and I published a new version of the package to Nuget. Thank you for your contribution!

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.

2 participants