-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Meet the Maintainer: Sudhanshu Dasgupta #6204
base: master
Are you sure you want to change the base?
Conversation
🚀 Preview for commit e6509a5 at: https://67939c7e3137ef1f86547a2a--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
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.
Needs work.
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.
Wrong font here.
<div class="interviewee"> | ||
<span>Sudhanshu:</span> | ||
<p> | ||
Actually there are a lot, so frequently we make release plans and at this moment a release plan is going for v0.9 of meshery. So there are a lot of areas that need help with and this is the best time to jump and take ownership of the field you feel comfortable with to complete. |
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.
Meshery should be capitalized.
<span>Sudhanshu:</span> | ||
<p> | ||
Actually there are a lot, so frequently we make release plans and at this moment a release plan is going for v0.9 of meshery. So there are a lot of areas that need help with and this is the best time to jump and take ownership of the field you feel comfortable with to complete. | ||
One important area I would say is which could be a hot topic to work on is metrics. |
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.
Grammatically incorrect.
<p> | ||
Actually there are a lot, so frequently we make release plans and at this moment a release plan is going for v0.9 of meshery. So there are a lot of areas that need help with and this is the best time to jump and take ownership of the field you feel comfortable with to complete. | ||
One important area I would say is which could be a hot topic to work on is metrics. | ||
We have two kinds of metrics: prometheus and grafana setup. So this area would need help because in the coming days we will be working to enhance this area so that users can easily connect their grafana or prometheus url and see metrics. |
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.
Capitalize prom and Graf.
<p> | ||
Actually there are a lot, so frequently we make release plans and at this moment a release plan is going for v0.9 of meshery. So there are a lot of areas that need help with and this is the best time to jump and take ownership of the field you feel comfortable with to complete. | ||
One important area I would say is which could be a hot topic to work on is metrics. | ||
We have two kinds of metrics: prometheus and grafana setup. So this area would need help because in the coming days we will be working to enhance this area so that users can easily connect their grafana or prometheus url and see metrics. |
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.
Missing commas.
</div> | ||
|
||
<div class="interviewee"> | ||
<span>Sudhanshu:</span> |
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.
@vishalvivekm, I thought that we had enhanced the Meet the Maintainers blog series to templatize interviewer and interviewee names, such they didn't have to be repeated other than in front matter. If we haven't, that's OK. If we have, we can use it here.
<span>Sudhanshu:</span> | ||
<p> | ||
Ok so there are many I am focusing on. One of them is of course Meshery itself. We are building the feature of multi-tenancy of environments and workspaces. The second is Sistent which is Layer5's own design system to migrate all components to this sistent. | ||
And the one is Layer5's own site which is running on gatsby so helping in getting prs reviewed there and merged and some dependency migration is also going on which i am focusing on. |
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.
"PRs"
"I"
<div class="interviewee"> | ||
<span>Sudhanshu:</span> | ||
<p> | ||
Ok so there are many I am focusing on. One of them is of course Meshery itself. We are building the feature of multi-tenancy of environments and workspaces. The second is Sistent which is Layer5's own design system to migrate all components to this sistent. |
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.
missing commas
<p> | ||
Hey Vivek, thank you for having me first of all. So I was in my junior year when I was also very eager to start and know about open source. | ||
I started with a few other projects but eventually I heard about Layer5 from my friend. | ||
So as a normal contributor 2 years from now, I tried to fix some issues like fixing some UI problems and all and then understand what's the goal of layer5 and about Meshery later on. When I learnt about Meshery then my hands couldn't wait to work on it since it was one of the great project in CNCF. So this continued and then I became an intern and followed by LFX(one of program run by Linux foundation) and eventually I became a maintainer. |
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.
"two years", not "2 years".
<div class="interviewee"> | ||
<span>Sudhanshu:</span> | ||
<p> | ||
Yes this is a great question. As a maintainer the first most important thing is the responsibility and success of the project. You are responsible for quality work that should be applied to the project. As a maintainer I try to provide quality reviews to the contributors and seek that they get their PRs merged. Being a maintainer is about building not just a tool or feature but taking ownership of the project's vision, leading development efforts, and mentoring new contributors to help them find their footing in open source. |
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.
missing commas.
Description
This PR fixes #
Notes for Reviewers
Signed commits