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.

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

Add Sicredi as adopter #4124

Merged
merged 7 commits into from
Jun 11, 2024
Merged

Add Sicredi as adopter #4124

merged 7 commits into from
Jun 11, 2024

Conversation

igorestevanjasinski
Copy link
Contributor

Adding Sicredi as an opentelemetry Adopter

@igorestevanjasinski igorestevanjasinski requested a review from a team March 11, 2024 12:42
Copy link

linux-foundation-easycla bot commented Mar 11, 2024

CLA Signed

The committers listed above are authorized under a signed CLA.

Comment on lines 79 to 80
reference: ''
contact: ''
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
reference: ''
contact: ''
reference: ''
contact: ''

Do you have any blog post, conference talk (recording, slides, etc.) we can provide as reference?

for the contact, please put here either your github handle (preferred) or an email address we can reach out to if we have questions, e.g. if any link throws 404s or if we plan to apply significant changes.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Hey Severin, we have only internal material, but I'm working to publish a blog post about it. Can I update the reference soon?

Copy link
Member

Choose a reason for hiding this comment

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

Hey Severin, we have only internal material, but I'm working to publish a blog post about it. Can I update the reference soon?

Sure! Just note that in that case we might need to wait a few days to merge this PR, we are currently in the progress of rewriting our requirements to be published on that list, which states right now that you need to have some reference material.

data/ecosystem/adopters.yaml Outdated Show resolved Hide resolved
@svrnm
Copy link
Member

svrnm commented Mar 14, 2024

Blocked by #4133

Let's finish the discussion on how we list adopters in the future, and then merge this one in.

@svrnm svrnm added the blocked label Mar 14, 2024
@svrnm svrnm removed the blocked label Jun 11, 2024
@svrnm
Copy link
Member

svrnm commented Jun 11, 2024

@igorestevanjasinski apologies for the delay here, the blocker is resolved and I will merge this PR now

@svrnm svrnm merged commit 77eee3d into open-telemetry:main Jun 11, 2024
16 checks passed
@igorestevanjasinski
Copy link
Contributor Author

@igorestevanjasinski apologies for the delay here, the blocker is resolved and I will merge this PR now

no problem, thank you @svrnm

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.

2 participants