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

Remove opentelemetry-proto in .gitsubmodule #5267

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Remove opentelemetry-proto in .gitsubmodule #5267

wants to merge 3 commits into from

Conversation

YHM404
Copy link

@YHM404 YHM404 commented Apr 26, 2024 •
edited

opentelemetry-proto was removed in #1713

@linux-foundation-easycla Linux Foundation: EasyCLA
Copy link

linux-foundation-easycla bot commented Apr 26, 2024 •
edited

CLA Signed

The committers listed above are authorized under a signed CLA.

  • ✅ login: YHM404 / name: ZOt6 (162bb40)
  • ✅ login: MrAlias / name: Tyler Yahn (997a7c8)
  • ✅ login: hanyuancheung / name: Chester Cheung (a2a4bcc)

@pellared
Copy link
Member

  1. Can you sign EasyCLA?
  2. I think you referenced a wrong PR.

@pellared pellared added the blocked:CLA Waiting on CLA to be signed before progress can be made label Apr 26, 2024
@MrAlias MrAlias added the response needed Waiting on user input before progress can be made label May 2, 2024
@pellared
Copy link
Member

@YHM404 bump

@YHM404
Copy link
Author

YHM404 commented May 21, 2024

  1. Can you sign EasyCLA?
  2. I think you referenced a wrong PR.

Hi, I already signed EasyCLA. and I recorrect the referenced PR.

@MrAlias MrAlias added Skip Changelog PRs that do not require a CHANGELOG.md entry and removed blocked:CLA Waiting on CLA to be signed before progress can be made response needed Waiting on user input before progress can be made labels May 21, 2024
pellared
pellared reviewed May 22, 2024
View reviewed changes
.gitmodules
Copy link
Member

@pellared pellared May 22, 2024 •
edited

Choose a reason for hiding this comment

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

Any reason for not simply removing the file?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@pellared pellared pellared approved these changes

@dmathieu dmathieu dmathieu approved these changes

@MrAlias MrAlias MrAlias approved these changes

@hanyuancheung hanyuancheung hanyuancheung approved these changes

@Aneurysm9 Aneurysm9 Awaiting requested review from Aneurysm9

@evantorrie evantorrie Awaiting requested review from evantorrie evantorrie is a code owner

@XSAM XSAM Awaiting requested review from XSAM XSAM is a code owner

@dashpole dashpole Awaiting requested review from dashpole dashpole is a code owner

@MadVikingGod MadVikingGod Awaiting requested review from MadVikingGod MadVikingGod is a code owner

Assignees
No one assigned
Labels
Skip Changelog PRs that do not require a CHANGELOG.md entry
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
5 participants
@YHM404 @pellared @dmathieu @MrAlias @hanyuancheung