Skip to content

Remove metrics from 1.0.0 release#1743

Merged
cijothomas merged 3 commits intoopen-telemetry:mainfrom
cijothomas:cijothomas/removemetrics
Jan 29, 2021
Merged

Remove metrics from 1.0.0 release#1743
cijothomas merged 3 commits intoopen-telemetry:mainfrom
cijothomas:cijothomas/removemetrics

Conversation

@cijothomas
Copy link
Member

@cijothomas cijothomas commented Jan 29, 2021

Metrics will release separately as a pre-release (like 1.1.0-alpha1), from a separate branch.

Immediate next steps after this PR merged:
create metrics branch off master, and revert this PR in that branch.
git tag metrics branch as 1.1.0-alpha1
modify MyGet daily publish GH action to run for both master,metrics branch. (this will publish 1.0.0-RC1 series packages from master, 1.1.0-alpha1 series from metrics branch and has metrics code)

reroute the in-flight metric PRs to metrics branch.

See #1501 for overall metrics plan.

@cijothomas cijothomas requested a review from a team January 29, 2021 15:35
@codecov
Copy link

codecov bot commented Jan 29, 2021

Codecov Report

Merging #1743 (14c8220) into main (ccc191f) will decrease coverage by 0.38%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #1743      +/-   ##
==========================================
- Coverage   83.28%   82.89%   -0.39%     
==========================================
  Files         250      193      -57     
  Lines        6874     6005     -869     
==========================================
- Hits         5725     4978     -747     
+ Misses       1149     1027     -122     
Impacted Files Coverage Δ
src/OpenTelemetry/Sdk.cs 100.00% <ø> (ø)

Copy link
Member

@CodeBlanch CodeBlanch left a comment

Choose a reason for hiding this comment

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

LGTM

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.

3 participants