Skip to content

Rename observer result to observable result #2238

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.

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

Merged
merged 2 commits into from
Jan 4, 2022

Conversation

legendecas
Copy link
Member

Follow up of #1645

Changes

  1. Rename the remaining usages of "Observer" to "Observable".

Related issues:

  1. feat(api-metrics): remove observable types opentelemetry-js#2687 (comment)

@github-actions
Copy link

This PR was marked stale due to lack of activity. It will be closed in 7 days.

@github-actions github-actions bot added the Stale label Dec 31, 2021
@bogdandrutu bogdandrutu removed the Stale label Jan 4, 2022
@reyang reyang added area:api Cross language API specification issue editorial Editorial changes only (typos, changelog, ...). No content-related changes of any kind. spec:metrics Related to the specification/metrics directory labels Jan 4, 2022
@jmacd jmacd enabled auto-merge (squash) January 4, 2022 19:21
@jmacd jmacd merged commit 5d374f8 into open-telemetry:main Jan 4, 2022
@legendecas legendecas deleted the observable branch January 5, 2022 02:37
carlosalberto pushed a commit to carlosalberto/opentelemetry-specification that referenced this pull request Oct 31, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:api Cross language API specification issue editorial Editorial changes only (typos, changelog, ...). No content-related changes of any kind. spec:metrics Related to the specification/metrics directory
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants