Skip to content
This repository has been archived by the owner on Oct 12, 2022. It is now read-only.

[WIP] bump version to 2.8.1 #224

Closed
wants to merge 6 commits into from
Closed

Conversation

TimothyMothra
Copy link
Member

@TimothyMothra TimothyMothra commented Oct 10, 2018

test started failing because Context.Properties was marked Obsolete.

  • Log4Net was incorrectly using Context.Properties instead of telemetry.Properties. Fixed.
  • NLog had a concept of Global Properties. Changed to use Context.GlobalProperties.
  • Fixed some unnecessary type casting

See also:
microsoft/ApplicationInsights-dotnet#839
microsoft/ApplicationInsights-dotnet#820
microsoft/ApplicationInsights-dotnet#923

@TimothyMothra TimothyMothra changed the title bump version to 2.8.0 [WIP] bump version to 2.8.0 Oct 10, 2018
CHANGELOG.md Outdated Show resolved Hide resolved
@TimothyMothra TimothyMothra changed the title [WIP] bump version to 2.8.0 [WIP] bump version to 2.8.1 Oct 19, 2018
@TimothyMothra
Copy link
Member Author

DO NOT MERGE, tests are failing

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.

1 participant