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

Readme upgrade #338

Merged
merged 5 commits into from
May 25, 2022
Merged

Readme upgrade #338

merged 5 commits into from
May 25, 2022

Conversation

pavangudiwada
Copy link
Contributor

Modified the README with new

  • Robusta logo
  • Table of contents
  • Acknowledgments
  • and many more

I believe this can help Robusta. Let me know if there are things I should change..

@arikalon1 arikalon1 requested a review from aantn May 23, 2022 15:10
Copy link
Collaborator

@aantn aantn left a comment

Choose a reason for hiding this comment

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

Very nice. Left two comments.

README.md Outdated
* Twitter - [@RobustaDev](https://twitter.com/RobustaDev)
* LinkedIn - [robusta-dev](https://www.linkedin.com/company/robusta-dev/)
* Jobs - [jobs@robusta.dev ](mailto:jobs@robusta.dev)
* Commercial Support - [support@robusta.dev ](mailto:support@robusta.dev )
Copy link
Collaborator

Choose a reason for hiding this comment

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

Maybe repace "Commercial Support" with "Email Support" - I want people to know that they can email us even if they're not a paying customer.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes understood. Made the change

Robusta is a troubleshooting and automations platform for Kubernetes. It streamlines everything that happens **after** you deploy your application. It is somewhat like Zapier/IFTTT for devops, with an emphasis on prebuilt useful automations and not just "build your own".
<div align="center">
<p>
<a href="https://home.robusta.dev/" target="_blank">
Copy link
Collaborator

Choose a reason for hiding this comment

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

What do you think of reducing the height of the logo a little? It takes up a lot of sceenspace on desktop and pushes the content down below the fold.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Reduced the size significantly.

Copy link
Collaborator

@aantn aantn left a comment

Choose a reason for hiding this comment

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

Awesome, thanks

@aantn aantn enabled auto-merge (squash) May 24, 2022 12:07
@aantn
Copy link
Collaborator

aantn commented May 24, 2022

Hmm, we seem to have some issue with our github checks that are preventing this from meging. (It can't merge without those checks but those checks also aren't running.)

I'll try and have someone look into this tomorrow.

@aantn aantn disabled auto-merge May 25, 2022 16:45
@aantn aantn merged commit 2764117 into robusta-dev:master May 25, 2022
@aantn
Copy link
Collaborator

aantn commented May 25, 2022

Merged! Thanks @pavangudiwada

@VVIISS
Copy link

VVIISS commented Sep 15, 2022

nice

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.

3 participants