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.

Sign up for GitHub

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

Jump to bottom

3.0.0 Release #145

Merged
merged 32 commits into from
Aug 20, 2019
Merged

3.0.0 Release #145

merged 32 commits into from
Aug 20, 2019

Conversation

nblumhardt
Copy link
Member

@nblumhardt nblumhardt commented Jun 18, 2019 •
edited
Loading

DigiBanks99 and others added 27 commits February 7, 2019 15:30
@DigiBanks99
Upgraded Microsoft.Extensions.Logging NuGet packages to 2.0.0
b6abdac
@nblumhardt
Proof-of-concept AddProvider() infrastructure
7030b19
@nblumhardt
Lean a bit more on IoC to wire up the factory and providers
e94a240
@DigiBanks99
Removed all netcore1.3 references and dependent code. Assumed netcore…
5c7ee2a 
…2.0 dependency from here on out.
@nblumhardt
Use FormattedLogValues as the event state
1ffaa7b
@nblumhardt
Merge branch 'dev' into dev
17bc472
@nblumhardt
Dev version bump [skip ci]
507cc4b
@SimonCropp
move to current sdk
3c27e7e
@SimonCropp
image: Visual Studio 2019 Preview
cf57808
@SimonCropp
Update global.json
6c36af7
@nblumhardt
Merge pull request #138 from SimonCropp/patch-1
7332262 
move to current sdk
@merbla
Update NuGet API Key
31d7194
@nblumhardt
Merge branch 'dev' of https://github.com/serilog/serilog-extensions-l…
8a9e537 
…ogging into add-provider
@nblumhardt
Merge upstream/dev
ad31fe8
@nblumhardt
SerilogLogValues to clean up provision of TState; re-version as 3.0.0…
6846f43 
…-* and limit dependencies
@nblumhardt
Typos/Cleanup
1b3b885
@nblumhardt
Use an SDK version present on AppVeyor :-)
e11cf7e
@nblumhardt
LangVersion
c950770
@nblumhardt
Add usual level control params to WriteTo.Providers()
d2155ce
@nblumhardt
Merge pull request #132 from nblumhardt/add-provider
79127bd 
SerilogLoggerFactory and AddProvider() infrastructure
@nblumhardt
Merge branch 'dev' into dev
3ee6f65
@nblumhardt
Merge pull request #131 from DigiBanks99/dev
e85c32f 
Upgraded Microsoft.Extensions.Logging NuGet packages to 2.0.0
@nblumhardt
Remove some unused build-time constants
a9923ae
@nblumhardt
Keyboard fumble
0885a2a
@nblumhardt
Merge pull request #140 from nblumhardt/remove-constants
7ba9559 
Remove some unused build-time constants
@nblumhardt
Make the level mapping routines public
3f73c40
@nblumhardt
Merge pull request #141 from nblumhardt/public-level-mapping
29bdf6c 
Make the level mapping routines public
@nblumhardt nblumhardt mentioned this pull request Jun 18, 2019
Merged
@nblumhardt
Copy link
Member Author

Folks, any thoughts/feelings/perspectives we're missing on whether this should go out the door as-is? We've had the brakes on in case there were breaking changes in the ASP.NET Core 3.x series, but it doesn't seem like that's going to be the case, so we should probably get this all out in the wild :-)

@nblumhardt
Copy link
Member Author

Ideally before merging this we'll finish #142 and #148

adamchester
adamchester approved these changes Aug 19, 2019
View reviewed changes
Copy link
Member

@adamchester adamchester left a comment •
edited
Loading

Choose a reason for hiding this comment

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

LGTM

(So far...)

adamchester
adamchester reviewed Aug 19, 2019
View reviewed changes
appveyor.yml Outdated Show resolved Hide resolved
@nblumhardt
Copy link
Member Author

#142 is ready to go

@adamchester
Merge pull request #142 from nblumhardt/event-id-caching
11c3c07 
Cache low-numbered numeric event ids to reduce allocations
@nblumhardt nblumhardt changed the title [WIP] 3.0.0 Release 3.0.0 Release Aug 20, 2019
@nblumhardt nblumhardt merged commit 71235c0 into master Aug 20, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@adamchester adamchester adamchester approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@nblumhardt @adamchester @DigiBanks99 @SimonCropp @merbla