-
-
Notifications
You must be signed in to change notification settings - Fork 743
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
docs(community): update latest maintainers list #3291
docs(community): update latest maintainers list #3291
Conversation
WalkthroughThe changes involve modifications to the Changes
Possibly related PRs
Suggested labels
Suggested reviewers
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
✅ Deploy Preview for asyncapi-website ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
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.
Actionable comments posted: 0
🧹 Outside diff range and nitpick comments (2)
config/MAINTAINERS.json (2)
Line range hint
1012-1021
: LGTM with a suggestion: Consider adding missing fields for consistency.The addition of Vinit Shahdeo as a maintainer is generally correct. However, for consistency with other entries, consider adding the following fields if available:
slack
company
This will ensure all maintainer entries have a uniform structure.
Line range hint
1022-1031
: LGTM with issues to address: Incorrect twitter handle and missing fields.The addition of Anubhav Vats as a maintainer is generally correct, but there are two issues to address:
The twitter handle is set to "postmanlabs", which seems incorrect for an individual maintainer. Please verify and update with Anubhav's personal twitter handle if available.
For consistency with other entries, consider adding the following fields if available:
slack
company
These changes will ensure accuracy and maintain a uniform structure across all maintainer entries.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
📒 Files selected for processing (1)
- config/MAINTAINERS.json (1 hunks)
🧰 Additional context used
🔇 Additional comments (2)
config/MAINTAINERS.json (2)
Line range hint
323-334
: LGTM: New maintainer added correctly.The addition of Azeez Elegbede as a maintainer is well-structured and contains all the necessary information. The entry follows the established format and includes the appropriate repositories.
Line range hint
261-286
: Confirmed: "asyncapi-react" removed from Lukasz Gornicki's repos.The removal of the "asyncapi-react" repository from Lukasz Gornicki's list of repos has been correctly implemented as mentioned in the AI summary.
⚡️ Lighthouse report for the changes in this PR:
Lighthouse ran on https://deploy-preview-3291--asyncapi-website.netlify.app/ |
Updated Maintainers list is available and this PR introduces changes with latest information about Maintainers
Summary by CodeRabbit
New Features
Updates