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

Adding K8gb to sandbox #2095

Merged
merged 7 commits into from
May 13, 2021
Merged

Adding K8gb to sandbox #2095

merged 7 commits into from
May 13, 2021

Conversation

amye
Copy link
Contributor

@amye amye commented May 13, 2021

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, you need to pick the single best GitHub repository for your project, not a GitHub organization.
  • Is your project closed source or, if it is open source, does your project have at least 300 GitHub stars?
  • Have you picked the single best (existing) category for your project?
  • Does it follow the other guidelines from the new entries section?
  • Have you included a URL for your SVG or added it to hosted_logos and referenced it there?
  • Does your logo clearly state the name of the project/product and follow the other logo guidelines?
  • Does your project/product name match the text on the logo?
  • Have you verified that the Crunchbase data for your organization is correct (including headquarters and LinkedIn)?
  • ~15 minutes after opening the pull request, the CNCF-Bot will post the URL for your staging server. Have you confirmed that it looks good to you and then added a comment to the PR saying "LGTM"?

Amye Scavarda Perrin added 2 commits May 13, 2021 11:19
@amye amye mentioned this pull request May 13, 2021
22 tasks
@CNCF-Bot
Copy link
Contributor

Build failed because of:
Empty twitter for Rancher Federal (member): https://twitter.com/rancherfederal
No cached entry, and Valve Software (member) has issues with twitter: https://twitter.com/valveoficial, 404 - {"errors":[{"code":34,"message":"Sorry, that page does not exist."}]}
Empty twitter for EMQ Technologies (member): https://twitter.com/emqtt
Empty twitter for EMQ Technologies: https://twitter.com/emqtt
No cached entry, and Sosivio (member) has issues with twitter: https://twitter.com/SosivioLtd, 404 - {"errors":[{"code":34,"message":"Sorry, that page does not exist."}]}
Empty twitter for StackStorm: https://twitter.com/Stack_Storm
k8gb  has an empty or missing homepage_url

@CNCF-Bot
Copy link
Contributor

CNCF-Bot commented May 13, 2021

Deploy preview for landscape ready!

Built with commit e5fd10e

https://deploy-preview-2095--landscape.netlify.app

@amye
Copy link
Contributor Author

amye commented May 13, 2021

LGTM

Signed-off-by: Chris Aniszczyk <[email protected]>
@somaritane
Copy link

@amye , it looks like k8gb has been assigned to the Remote Procedure Call category, but it should be in the Coordination & Service Discovery instead.

Signed-off-by: Chris Aniszczyk <[email protected]>
@somaritane
Copy link

@amye , it looks like k8gb has been assigned to the Remote Procedure Call category, but it should be in the Coordination & Service Discovery instead.

@caniszczyk thanks a lot!

@somaritane
Copy link

@amye @caniszczyk, just checked, looks awesome!
Just one thing, I see that the k8gb project name is missing below the logo.
Is it possible to replace the k8gb icon logo variant with a stacked logo variant from artwork https://github.com/cncf/artwork/blob/master/projects/k8gb/stacked/color/k8gb-stacked-color.svg?
Thanks a lot!

@amye
Copy link
Contributor Author

amye commented May 13, 2021

0b8f815 resolves.
@somaritane, the landscape can be contributed to by anyone.

@somaritane
Copy link

0b8f815 resolves.
@somaritane, the landscape can be contributed to by anyone.

Thanks, @amye, good to know!

@caniszczyk caniszczyk merged commit 3f744fe into master May 13, 2021
@caniszczyk caniszczyk deleted the k8gb-sandbox branch May 13, 2021 21:06
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.

4 participants