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

Added maintainers column to backend projects #5088

Merged
merged 20 commits into from
Nov 17, 2023

Conversation

ankushbhardwaj408
Copy link
Member

Description

This PR fixes #5057

Notes for Reviewers

Signed commits

  • Yes, I signed my commits.

@l5io
Copy link
Contributor

l5io commented Oct 30, 2023

🚀 Preview for commit 1faf54e at: https://653fe3a92f843312ba33995d--layer5.netlify.app

@l5io
Copy link
Contributor

l5io commented Nov 4, 2023

🚀 Preview for commit 9f9b656 at: https://654614e7a334a965c8c3c612--layer5.netlify.app

@abhijeetgauravm
Copy link
Contributor

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.

Copy link
Contributor

@ritiksaxena124 ritiksaxena124 left a 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]>
@ankushbhardwaj408
Copy link
Member Author

@ritiksaxena124 Done.

@l5io
Copy link
Contributor

l5io commented Nov 5, 2023

🚀 Preview for commit 96045fa at: https://6547d89816594777825ea358--layer5.netlify.app

Copy link
Contributor

@iArchitSharma iArchitSharma left a comment

Choose a reason for hiding this comment

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

LGTM!

@l5io
Copy link
Contributor

l5io commented Nov 6, 2023

🚀 Preview for commit b5d621e at: https://6549222f3350cd1d91c122d7--layer5.netlify.app

Copy link
Member

@Mohith234 Mohith234 left a comment

Choose a reason for hiding this comment

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

Yay 🎈, LGTM @ankushbhardwaj408

Copy link
Member

@leecalcote leecalcote left a 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?

@ankushbhardwaj408
Copy link
Member Author

ankushbhardwaj408 commented Nov 7, 2023

@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

@iArchitSharma
Copy link
Contributor

@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?

@ankushbhardwaj408
Copy link
Member Author

@iArchitSharma Done.

@l5io
Copy link
Contributor

l5io commented Nov 7, 2023

🚀 Preview for commit 7905cd7 at: https://6549ffb261fe677640aa10a0--layer5.netlify.app

@l5io
Copy link
Contributor

l5io commented Nov 13, 2023

🚀 Preview for commit 3425bd8 at: https://6552508f9684d803e6ee8fed--layer5.netlify.app

@iArchitSharma
Copy link
Contributor

LGTM!

@@ -168,24 +173,28 @@ export const repo_data = [
project: "Cloud Native Labs",
Copy link
Member

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"],
Copy link
Member

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"],
Copy link
Member

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"],
Copy link
Member

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"],
Copy link
Member

@leecalcote leecalcote Nov 14, 2023

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"],
Copy link
Member

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...

Copy link
Member Author

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"],
Copy link
Member

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"],
Copy link
Member

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"],
Copy link
Member

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"],
Copy link
Member

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"],
Copy link
Member

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"],
Copy link
Member

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: [
Copy link
Member

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]>
@l5io
Copy link
Contributor

l5io commented Nov 15, 2023

🚀 Preview for commit 0fe13da at: https://6554487fd4c7e0572ef28aef--layer5.netlify.app

Signed-off-by: Ankush Bhardwaj <[email protected]>
@ankushbhardwaj408
Copy link
Member Author

@leecalcote I've created PRs to add maintainers.md layer5io/sistent#339 meshery/meshery#9365

@l5io
Copy link
Contributor

l5io commented Nov 15, 2023

🚀 Preview for commit 767073c at: https://65544dfc4bbe7259d05f3357--layer5.netlify.app

@l5io
Copy link
Contributor

l5io commented Nov 16, 2023

🚀 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"],
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
maintainers_name: ["Lee Calcote"],
maintainers_name: ["Vacant"],

@@ -41,26 +41,29 @@ export const repo_data = [
image: meshery,
site: "https://meshery.io",
language: "Jekyll",
maintainers_name: ["Vacant"],
link: ["#"],
maintainers_name: ["Nikhil Ladha"],
Copy link
Member

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.

Copy link
Member Author

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]>
@l5io
Copy link
Contributor

l5io commented Nov 16, 2023

🚀 Preview for commit 7a1c9b0 at: https://6555a3a243d1113b3204625a--layer5.netlify.app

@leecalcote leecalcote merged commit fe2464f into layer5io:master Nov 17, 2023
4 checks passed
@leecalcote
Copy link
Member

Thank you, @ankushbhardwaj408 👏

@ankushbhardwaj408
Copy link
Member Author

Thanks to you @leecalcote for your patience and all the suggestions. Happy to contribute!🫡

@ankushbhardwaj408 ankushbhardwaj408 deleted the ankushbhardwaj408-#5057 branch November 20, 2023 04:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

[Community] Handbook: Add maintainer column to backend projects table
7 participants