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

.NET Core refactor #7

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

.NET Core refactor #7

wants to merge 8 commits into from

Conversation

bgradin
Copy link
Member

@bgradin bgradin commented Dec 18, 2017

.NET Core Refactor

Goal: To get Toolhouse.Monitoring working with .NET Core, in order to facilitate development of ASP.NET Core projects
Framework Targets: .NET v4.5.1, .NET v4.6, .NET v4.6.1, .NET Core v2.0

Status: Tested in .NET 4.6.1 and .NET Core v2.0, no errors found

API Changes

Non-ASP.NET Core

For non-ASP.NET Core projects, the API stays completely the same, except for one small change: Instead of creating a database dependency with:

Readiness.AddDatabaseDependency("MyConnection")

you use:

Readiness.AddDatabaseDependency(ConfigurationManager.ConnectionStrings["MyConnection"].ConnectionString)

ASP.NET Core

See the sections added to the docs for information about the new ASP.NET Core API

This hasn't been tested yet; there will likely be bug fixes one it has been.
This time everything is tested and works, and a nuget package is being generated.
@bgradin bgradin force-pushed the feature/aspnetcore branch 5 times, most recently from ac1cbf0 to 25df6df Compare December 19, 2017 22:45
@bgradin bgradin requested review from matthinz and tantalic and removed request for matthinz December 19, 2017 22:50
@bgradin bgradin changed the title WIP: ASP.NET Core refactor .NET Core refactor Dec 19, 2017
@bgradin bgradin requested a review from matthinz December 19, 2017 23:08
@bgradin bgradin force-pushed the feature/aspnetcore branch from d697d70 to 026e89d Compare March 12, 2018 22:54
…aspnetcore

# Conflicts:
#	.travis.yml
#	Toolhouse.Monitoring.Tests/Toolhouse.Monitoring.Tests.csproj
#	Toolhouse.Monitoring/Properties/AssemblyInfo.cs
#	Toolhouse.Monitoring/Toolhouse.Monitoring.csproj
@bgradin bgradin force-pushed the feature/aspnetcore branch from 026e89d to 208c800 Compare March 13, 2018 18:24
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