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

[chore] [cmd/mdatagen] Remove redis todo for missing units #24384

Merged
merged 2 commits into from
Jul 20, 2023
Merged

[chore] [cmd/mdatagen] Remove redis todo for missing units #24384

merged 2 commits into from
Jul 20, 2023

Conversation

mackjmr
Copy link
Member

@mackjmr mackjmr commented Jul 19, 2023

Description:
Now that #23573 has been merged, we can remove the todo.

@mackjmr mackjmr requested a review from a team July 19, 2023 08:18
@mackjmr mackjmr requested a review from dmitryax as a code owner July 19, 2023 08:18
@github-actions github-actions bot added the cmd/mdatagen mdatagen command label Jul 19, 2023
@songy23 songy23 added the ready to merge Code review completed; ready to merge by maintainers label Jul 20, 2023
@dmitryax dmitryax merged commit 6389064 into open-telemetry:main Jul 20, 2023
@github-actions github-actions bot added this to the next release milestone Jul 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@atoulme atoulme atoulme approved these changes

@dmitryax dmitryax Awaiting requested review from dmitryax

Assignees

@fatsheep9146 fatsheep9146

Labels
cmd/mdatagen mdatagen command ready to merge Code review completed; ready to merge by maintainers
Projects
None yet
Milestone
v0.82.0
Development

Successfully merging this pull request may close these issues.
5 participants
@mackjmr @atoulme @dmitryax @songy23 @fatsheep9146