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

Init LlamaIndex skeleton #19038

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

yahya-mouman
Copy link
Contributor

@yahya-mouman yahya-mouman commented Nov 12, 2024

What does this PR do?

This PR initializes the skeleton for an upcoming LlamaIndex integration.
This is not meant to be published or synced. And is just an empty skeleton for the upcoming integration.
https://datadoghq.atlassian.net/browse/MLOB-1881

Motivation

In order to get an official logo for Llama index from the design org we needed to provide an app-id and uuid generated in this PR.

Review checklist (to be filled by reviewers)

  • Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
  • Add the qa/skip-qa label if the PR doesn't need to be tested during QA.
  • If you need to backport this PR to another branch, you can add the backport/<branch-name> label to the PR and it will automatically open a backport PR once this one is merged

Copy link

The validations job has failed; please review the Files changed tab for possible suggestions to resolve.

Copy link

The validations job has failed; please review the Files changed tab for possible suggestions to resolve.

Copy link

The validations job has failed; please review the Files changed tab for possible suggestions to resolve.

@maycmlee
Copy link
Contributor

maycmlee commented Nov 12, 2024

This is not meant to be published or synced. And is just an empty skeleton for the upcoming integration.

Hi @yahya-mouman! Since that's the case, I'll go ahead and remove documentation from the reviewer's list. If anything changes and you need us to review, feel free to add us back.

Copy link
Contributor

@maycmlee maycmlee left a comment

Choose a reason for hiding this comment

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

No review needed. I couldn't remove documentation from the reviewer's list 🙃 so am commenting so that I can remove it from our PR queue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants