Skip to content

Conversation

@asyncapi-bot
Copy link
Contributor

@asyncapi-bot asyncapi-bot commented Oct 15, 2025

Updated Ambassadors list is available and this PR introduces changes with latest information about Ambassadors list

Summary by CodeRabbit

  • New Features
    • Added a new Ambassador profile: Ankit Kumar. Includes bio, company and title, country flag, profile image, and social links.
    • Showcases multiple contributions: talks, booth participation, and articles with titles, dates, and links.
    • Enhances the Ambassadors listing with richer contributor details and recent activity.

@netlify
Copy link

netlify bot commented Oct 15, 2025

Deploy Preview for asyncapi-website failed.

Name Link
🔨 Latest commit 6e589a4
🔍 Latest deploy log https://app.netlify.com/projects/asyncapi-website/deploys/68ef4d5862c41c0008bde418

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Oct 15, 2025

Walkthrough

Adds a new ambassador entry for Ankit Kumar to config/AMBASSADORS_MEMBERS.json, including profile fields (name, GitHub, IDs, social links, company, title, bio, image) and a contributions array with talks, booth activity, and articles. No other files or existing entries were changed.

Changes

Cohort / File(s) Summary
Ambassadors roster update
config/AMBASSADORS_MEMBERS.json
Appends a new Ambassador object for Ankit Kumar with full metadata (githubID, slack, linkedin, country, bio, company, title, img) and six contributions (talks, booth, articles) with dates and links.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

Suggested reviewers

  • derberg
  • akshatnema
  • anshgoyalevil
  • sambhavgupta0705
  • Mayaleeeee
  • asyncapi-bot-eve

Poem

A carrot of code, a nibble of JSON delight,
I thump the ground—“New ambassador in sight!”
Talks and articles hop in a neat array,
Booth tales sparkle like dewdrops in May.
Ears up, paws down—commit squeaks “all right!”
Hop, save, merge—ship it tonight! 🥕✨

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Title Check ⚠️ Warning The pull request title refers to updating the TSC members list, but the changes actually update the ambassadors list by adding a new ambassador entry, causing a mismatch between title and content. Please update the title to accurately reflect that the PR updates the ambassadors list, for example “docs(community): update latest ambassadors list” or “docs(community): add Ankit Kumar to ambassadors list.”
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage ✅ Passed No functions found in the changes. Docstring coverage check skipped.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch update-ambassadors-61aa1e708849d85a4dababae09ebd7b9b2968c31

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@codecov
Copy link

codecov bot commented Oct 15, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.00%. Comparing base (568740e) to head (6e589a4).

Additional details and impacted files
@@            Coverage Diff            @@
##            master     #4478   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           22        22           
  Lines          778       778           
  Branches       144       144           
=========================================
  Hits           778       778           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Copy link
Contributor

@coderabbitai coderabbitai bot left a 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

🧹 Nitpick comments (2)
config/AMBASSADORS_MEMBERS.json (2)

755-766: New ambassador entry looks good; confirm TSC status and field norms

  • Please confirm if Ankit Kumar should be marked as a TSC member per PR title. If yes, add "isTscMember": true.
  • Optional: align linkedin format (slug vs full URL) with project conventions.

786-793: Normalize contribution type "booth" to avoid schema/UX mismatch

Past entries use "special contribution" for booth duty. If the site expects a fixed enum, "booth" may break filters/rendering. Recommend aligning:

-            {
-                "type": "booth",
+            {
+                "type": "special contribution",
                 "title": "Manned the AsyncAPI booth during APIDays Singapore to connect with developers and promote the AsyncAPI initiative.",
                 "date": {
                     "year": 2025,
                     "month": "April"
                 },
                 "link": ""
             },
📜 Review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 568740e and 6e589a4.

📒 Files selected for processing (1)
  • config/AMBASSADORS_MEMBERS.json (1 hunks)
⏰ Context from checks skipped due to timeout of 180000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (3)
  • GitHub Check: Automerge PR autoapproved by a bot
  • GitHub Check: Test NodeJS PR - windows-latest
  • GitHub Check: Lighthouse CI

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants