-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Added maintainers column to backend projects #5088
Added maintainers column to backend projects #5088
Conversation
Signed-off-by: Ankush Bhardwaj <[email protected]>
🚀 Preview for commit 1faf54e at: https://653fe3a92f843312ba33995d--layer5.netlify.app |
🚀 Preview for commit 9f9b656 at: https://654614e7a334a965c8c3c612--layer5.netlify.app |
Hey @ankushbhardwaj408 Let's discuss this on website call tomorrow at 6:30 PM IST / 7 AM Central time. Please add this as an agenda item in the meeting minutes. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Merge Conflicts
Signed-off-by: Ankush Bhardwaj <[email protected]>
@ritiksaxena124 Done. |
🚀 Preview for commit 96045fa at: https://6547d89816594777825ea358--layer5.netlify.app |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
🚀 Preview for commit b5d621e at: https://6549222f3350cd1d91c122d7--layer5.netlify.app |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yay 🎈, LGTM @ankushbhardwaj408
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Will you please undo any changes not directly related to the issue?
@leecalcote Thank you for your input. I have carefully reviewed the changes, and there are no changes that are not related to the problem, if there are still any please let me know |
@ankushbhardwaj408 I think Lee is referring to the code formatting here. Can you keep the formatting as it was? |
Signed-off-by: Ankush Bhardwaj <[email protected]>
Signed-off-by: Ankush Bhardwaj <[email protected]>
Signed-off-by: Ankush Bhardwaj <[email protected]>
@iArchitSharma Done. |
🚀 Preview for commit 7905cd7 at: https://6549ffb261fe677640aa10a0--layer5.netlify.app |
🚀 Preview for commit 3425bd8 at: https://6552508f9684d803e6ee8fed--layer5.netlify.app |
LGTM! |
@@ -168,24 +173,28 @@ export const repo_data = [ | |||
project: "Cloud Native Labs", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please change this to Cloud Native Learning Paths (https://layer5.io/learn/learning-paths)
repository: | ||
"https://github.com/meshery/play", | ||
} | ||
maintainers_name: ["Nikhil Ladha"], |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please leave maintainer vacant.
@@ -228,37 +239,56 @@ export const repo_data = [ | |||
project: "Meshery", | |||
image: meshery, | |||
language: "Golang, gRPC", | |||
description: "the cloud native management plane to provide governance and backend system integration. Golang, gRPC, NATS, CloudEvents", | |||
maintainers_name: ["Nikhil Ladha", "Aaditya Narayan Subedy"], |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please swap with @MUzairS15 and @nebula-aac
repository: "https://github.com/meshery/meshery-operator", | ||
}, | ||
{ | ||
project: "MeshSync", | ||
image: MeshSync, | ||
language: "Golang and NATS", | ||
description: "MeshSync is a multi-mesh resource discovery application which is also a custom kubernetes controller.", | ||
maintainers_name: ["Vacant"], |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please add @theBeginner86
@@ -284,14 +314,20 @@ export const repo_data = [ | |||
project: "Image Hub", | |||
image: imagehub, | |||
language: "Vuejs", | |||
description: "A sample application written to run on Istio or Consul for exploring WebAssembly modules used as Envoy filters.", | |||
maintainers_name: ["Lee Calcote"], |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please leave vacant.
description: "meshery-perf is a Meshery component used to run performance tests", | ||
repository: "https://github.com/meshery/meshery-perf" | ||
} | ||
maintainers_name: ["Vacant"], |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@gyohuangxin? - - he'll have to confirm...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'll change it after @gyohuangxin confirms.
@@ -359,6 +417,12 @@ export const repo_data = [ | |||
project: "Cloud Native Patterns", | |||
image: patterns, | |||
language: "Yaml", | |||
maintainers_name: ["Lee Calcote", "Nic Jackson", "Nikhil Ladha"], |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remove Nikhil.
description: "Meshery adapter for management of Network Service Mesh.", | ||
repository: "https://github.com/layer5io/meshery-nsm", | ||
}, | ||
{ | ||
project: "Meshery Adapter for Octarine", | ||
image: meshery, | ||
language: "Golang, gRPC", | ||
maintainers_name: ["Lee Calcote"], |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please move this Adapter altogether. It has been archived.
description: "Meshery adapter for management of AWS App Mesh", | ||
repository: "https://github.com/layer5io/meshery-app-mesh", | ||
}, | ||
{ | ||
project: "Meshery Adapter for Tanzu Service Mesh", | ||
image: meshery, | ||
language: "Golang, gRPC", | ||
description: "Meshery adapter for management of VMware Tanzu Service Mesh", | ||
maintainers_name: ["Xin Huang"], |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should be @hexxdump
description: "Meshery adapter for management of NGINX Service Mesh", | ||
repository: "https://github.com/layer5io/meshery-nginx-sm", | ||
}, | ||
{ | ||
project: "Meshery Adapter for Cilium Service Mesh", | ||
image: ciliumLogo, | ||
language: "Golang, gRPC", | ||
maintainers_name: ["Xin Huang"], |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Switch to @MUzairS15
repository: "https://github.com/layer5io/meshery-adapter-library", | ||
}, | ||
{ | ||
project: "Meshery Adapter Template", | ||
image: meshery, | ||
language: "Golang, gRPC", | ||
description: "A GitHub repo used as a template for new Meshery Adapters.", | ||
maintainers_name: ["Antonette Caldwell"], |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please leave vacant.
@@ -195,29 +204,31 @@ export const repo_data = [ | |||
project: "Linkerd Service Mesh Workshop", | |||
image: linkerdLogo, | |||
language: "Meshery", | |||
maintainers_name: ["Hussaina Begum"], |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please leave vacant.
@@ -304,6 +340,18 @@ export const repo_data = [ | |||
project: "Cloud Native Performance", | |||
image: smp, | |||
language: "Protobuf", | |||
maintainers_name: [ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please add @asubedy
Signed-off-by: Ankush Bhardwaj <[email protected]>
Signed-off-by: Ankush Bhardwaj <[email protected]>
🚀 Preview for commit 0fe13da at: https://6554487fd4c7e0572ef28aef--layer5.netlify.app |
Signed-off-by: Ankush Bhardwaj <[email protected]>
@leecalcote I've created PRs to add maintainers.md layer5io/sistent#339 meshery/meshery#9365 |
🚀 Preview for commit 767073c at: https://65544dfc4bbe7259d05f3357--layer5.netlify.app |
🚀 Preview for commit 3e2eeba at: https://6555965f0030203272b79370--layer5.netlify.app |
description: "Meshery adapter for management of Consul", | ||
repository: "https://github.com/layer5io/meshery-consul", | ||
}, | ||
{ | ||
project: "Meshery Adapter for CPX", | ||
image: meshery, | ||
language: "Golang, gRPC", | ||
maintainers_name: ["Lee Calcote"], |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
maintainers_name: ["Lee Calcote"], | |
maintainers_name: ["Vacant"], |
Signed-off-by: Lee Calcote <[email protected]>
@@ -41,26 +41,29 @@ export const repo_data = [ | |||
image: meshery, | |||
site: "https://meshery.io", | |||
language: "Jekyll", | |||
maintainers_name: ["Vacant"], | |||
link: ["#"], | |||
maintainers_name: ["Nikhil Ladha"], |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Change to Vivek Vishal, please.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@leecalcote Done.
Signed-off-by: Ankush Bhardwaj <[email protected]>
🚀 Preview for commit 7a1c9b0 at: https://6555a3a243d1113b3204625a--layer5.netlify.app |
Thank you, @ankushbhardwaj408 👏 |
Thanks to you @leecalcote for your patience and all the suggestions. Happy to contribute!🫡 |
Description
This PR fixes #5057
Notes for Reviewers
Signed commits