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 Sedai (try 2) #275

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Add Sedai (try 2) #275

wants to merge 1 commit into from

Conversation

johnjamie
Copy link
Contributor

Pre-submission checklist:

Please check each of these after submitting your pull request:

  • Are you only including a repo_url if your project is 100% open source? If so, pick the single best GitHub repository for your project, not a GitHub organization.
  • Does it follow the guidelines for new entries?
  • Have you added an SVG to hosted_logos and referenced it there?
  • Does your logo clearly state the name of the project/product and follow logo guidelines?
  • Have you verified that the Crunchbase data for your organization is correct (including headquarters and LinkedIn)?
  • ~5 minutes after opening the pull request, the CNCF-Bot will post a link for you to preview your changes. Have you confirmed that they are correct and then added a comment to the PR saying "LGTM"?

@netlify
Copy link

netlify bot commented Nov 9, 2022

Deploy Preview for cdf-landscape failed.

Name Link
🔨 Latest commit 7cb610d
🔍 Latest deploy log https://app.netlify.com/sites/cdf-landscape/deploys/636c2b9cb170d3000814b369

- item:
name: Sedai
description: >-
Sedai optimizes & implements configurations for containerized and serverless applications autonomously based on ML-based analysis of production behavior
Copy link
Member

Choose a reason for hiding this comment

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

Please make sure to indent the description correctly - 2 spaces. You can check others with the description as examples.

@fdegir
Copy link
Member

fdegir commented Nov 9, 2022

@johnjamie In addition to the indentation issue I mentioned inline, it looks like Sedai logo was removed by the landscape bot, probably due to missing association/use of the logo on the landscape.

Could you please make sure you add Sedai logo into hosted_logos + add Sedai entry to landscape.yml in the same pull request?

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.

None yet

2 participants