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

Support for .NET Core 3.1 #107

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

soundarmoorthy
Copy link

No description provided.

Plain Elastic is migrated to .NET Standard
Tests are now in .NET Core
There are missing extension methods from latest dependencies. Wrote them
on my own. This is primarily for testing and not for the core library.
This is how the Core project is created
* Open the solution in VS for Mac
* Create a new .NET Standard project under the solution
* use `git mv` command to move the files from .NET Fx project to .NET
standard project.
* Similarly do the same for other projects.
* Test project is based on NUnit framework primarily.
* Update the Newtonsoft.JSON and other dependencies to latest
…empt created a .NET Core application instead of a test project, so the tests were not detected by the IDE at all
…ct to facilitate running tests from IDE. Now it works from within Visual Studio for Mac
…called on a null object the test fails. The fix is to check for nulls before invoking methods on an object. If both object and it's comparison are null return true, if object is null and comparison is not return false. If both are not null do the actual equals
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.

1 participant