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

Update RELEASING.md with correct tag.sh command #130

Merged
merged 1 commit into from
Jul 10, 2020
Merged

Update RELEASING.md with correct tag.sh command #130

merged 1 commit into from
Jul 10, 2020

Conversation

MrAlias
Copy link
Contributor

@MrAlias MrAlias commented Jul 10, 2020

No description provided.

@MrAlias MrAlias added the documentation Improvements or additions to documentation label Jul 10, 2020
@MrAlias MrAlias self-assigned this Jul 10, 2020
@MrAlias MrAlias merged commit 70d36d9 into open-telemetry:master Jul 10, 2020
@MrAlias MrAlias deleted the RELEASING branch July 10, 2020 18:30
YANYZP pushed a commit to YANYZP/opentelemetry-go-contrib that referenced this pull request Jul 20, 2020
@MrAlias
Update RELEASING.md with correct tag.sh cmd (open-telemetry#130)
eb4ef69
@MrAlias MrAlias mentioned this pull request Jul 20, 2020
Merged
YANYZP pushed a commit to YANYZP/opentelemetry-go-contrib that referenced this pull request Jul 23, 2020
@MrAlias
Update RELEASING.md with correct tag.sh cmd (open-telemetry#130)
c026732
plantfansam referenced this pull request in plantfansam/opentelemetry-go-contrib Mar 18, 2022
@krnowak @rghetia
Drop the registry package (#130)
3362421 
This is to shrink the PR #100.

The only place where the registry.Variable type was used was metrics,
so just inline that type into its only user. The use of the
registry.Variable type in core.Key was limited to the Name field.

The stats package also used the registry.Variable type, but seems that
also only the Name field was used and the package is going to be
dropped anyway.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Aneurysm9 Aneurysm9 Aneurysm9 approved these changes

@evantorrie evantorrie Awaiting requested review from evantorrie

@jmacd jmacd Awaiting requested review from jmacd

@lizthegrey lizthegrey Awaiting requested review from lizthegrey

@paivagustavo paivagustavo Awaiting requested review from paivagustavo

Assignees

@MrAlias MrAlias

Labels
documentation Improvements or additions to documentation
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@MrAlias @Aneurysm9