Skip to content

Add native support for Asp.net core #14

Merged
merged 11 commits into from
May 22, 2019
Merged

Add native support for Asp.net core #14

merged 11 commits into from
May 22, 2019

Conversation

Bobreshovr
Copy link
Contributor

#1

@codecov-io
Copy link

codecov-io commented May 21, 2019

Codecov Report

Merging #14 into master will increase coverage by 2.22%.
The diff coverage is 56.33%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #14      +/-   ##
==========================================
+ Coverage   56.04%   58.27%   +2.22%     
==========================================
  Files          25       28       +3     
  Lines         728      798      +70     
==========================================
+ Hits          408      465      +57     
- Misses        320      333      +13
Impacted Files Coverage Δ
src/Horarium/HorariumServer.cs 30.43% <0%> (+30.43%) ⬆️
src/Horarium.AspNetCore/HorariumLogger.cs 25% <25%> (ø)
src/Horarium.AspNetCore/JobFactory.cs 40% <40%> (ø)
...rarium.AspNetCore/RegistrationHorariumExtension.cs 76.19% <76.19%> (ø)
src/Horarium/HorariumClient.cs 28.57% <0%> (+28.57%) ⬆️
src/Horarium/Handlers/StatisticsJobs.cs 57.14% <0%> (+57.14%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c931361...fd1423d. Read the comment docs.

@Bobreshovr Bobreshovr changed the title Feature 1 Add native support for Asp.net core May 21, 2019
README.md Outdated
public void ConfigureServices(IServiceCollection services)
{
//...
service.AddHorariumServer(MongoRepositoryFactory.Create("mongodb://localhost:27017/horarium"));
Copy link
Contributor

Choose a reason for hiding this comment

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

serviceS

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done

@Bobreshovr Bobreshovr merged commit c969ebf into master May 22, 2019
}
}
}
}
Copy link
Contributor

Choose a reason for hiding this comment

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

new line

}
}
}
}
Copy link
Contributor

Choose a reason for hiding this comment

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

new line

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants