FIX: Enrich context before collecting properties

[purplepangolin]

LogCompletion event was not including properties added via
EnrichDiagnosticContext delegate on options because
the properties had been collected before invoking the enricher.

Swapping the order of the statements fixes this.

This is a fix for #148

I'm afraid I haven't had time to add a test for this. I have built the package locally and tested it with the project I am currently working on.

[nblumhardt]

Oops! Nice catch, thanks 👍

[D9001]

Hi, I was following some blog posts and set up my app to use EnrichDiagnosticContext from within UseSerilogRequestLogging in a .NET Core 2.0 library. This made my app logging so much cleaner.
However, I just realized that I only have access to these when I use I use one of the latest dev versions of the nuget package. Is there going to be a stable release available soon which makes use of these? Just wanted to make sure before I roll back my changes in time for our production release.

[nblumhardt]

@D9001 I'll set up a WIP release PR now so that you can track it, thanks; sounds like you're happy with the shape of EnrichDiagnostiContext as it's implemented in the previews?

[D9001]

@nblumhardt Absolutely. Thanks!