Skip to content

Conversation

@cijothomas
Copy link
Member

This PR is adding a MinVerTagPreix of "core-" to all the core projects. "Core" projects refers to the one which has an official spec https://github.com/open-telemetry/opentelemetry-specification currently marked "Stable". These projects are intended to be released as 1.0.0 soon.

Rest of the projects (instrumentation, extensions, opentracing shim etc) will remain in RC state, until their respective spec is marked stable and/or the project itself stabilizes.

(though it is releasing separately initially, the medium term we'd expect to release everything together with same version. and in long term, the instrumentations may even move out of this repo to elsewhere, likely current contrib repo or new dedicated per instrumentation repos.)

@cijothomas cijothomas requested a review from a team February 2, 2021 18:01
@codecov
Copy link

codecov bot commented Feb 2, 2021

Codecov Report

Merging #1774 (4e3d9a8) into main (c141779) will increase coverage by 0.01%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #1774      +/-   ##
==========================================
+ Coverage   84.17%   84.19%   +0.01%     
==========================================
  Files         187      187              
  Lines        5934     5934              
==========================================
+ Hits         4995     4996       +1     
+ Misses        939      938       -1     
Impacted Files Coverage Δ
...ZPages/Implementation/ZPagesExporterEventSource.cs 62.50% <0.00%> (+6.25%) ⬆️

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

@cijothomas cijothomas merged commit be5efda into main Feb 2, 2021
@cijothomas cijothomas deleted the cijothomas/versionseparationforcore branch February 2, 2021 19:25
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.

4 participants