Skip to content

chore: add twitter handle for @ankitk-me#2100

Merged
asyncapi-bot merged 1 commit intoasyncapi:masterfrom
ankitk-me:ambassadors-build-fix
Oct 15, 2025
Merged

chore: add twitter handle for @ankitk-me#2100
asyncapi-bot merged 1 commit intoasyncapi:masterfrom
ankitk-me:ambassadors-build-fix

Conversation

@ankitk-me
Copy link
Contributor

@ankitk-me ankitk-me commented Oct 15, 2025

Description

  • Add twitter handle for @ankitk-me in ambassadors list.

Summary by CodeRabbit

  • New Features
    • Added a Twitter handle to Ankit Kumar’s ambassador profile, enabling direct access to his Twitter from the profile.
    • The Twitter icon/link now appears wherever ambassador social links are shown, including listings and profile detail pages.
    • Improves discoverability and engagement by providing an additional contact channel directly within the ambassador’s visible profile.

@coderabbitai
Copy link

coderabbitai bot commented Oct 15, 2025

Walkthrough

Added a twitter field ("ankitm_me") to Ankit Kumar’s entry in AMBASSADORS_MEMBERS.yaml.

Changes

Cohort / File(s) Summary
Ambassador metadata update
AMBASSADORS_MEMBERS.yaml
Added twitter: "ankitm_me" to Ankit Kumar’s record.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

Suggested labels

ready-to-merge

Suggested reviewers

  • thulieblack

Poem

I twitch my nose at YAML rows,
A tiny field, a handle shows—
@ankitm_me now hops in view,
The carrot of commits, bright and new.
Thump-thump! The config gardens sing,
One more link on social wing. 🥕🐇

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title Check ✅ Passed The title clearly and concisely describes the primary change—adding a Twitter handle for the ambassador entry @ankitk-me—and uses the “chore” prefix appropriately to indicate a non-functional update, making it easy for teammates to understand the main purpose of the PR at a glance.
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

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 61aa1e7 and 12c2f9b.

📒 Files selected for processing (1)
  • AMBASSADORS_MEMBERS.yaml (1 hunks)
🔇 Additional comments (1)
AMBASSADORS_MEMBERS.yaml (1)

531-531: LGTM; please just verify the handle is correct and style-consistent.

Formatting and placement match the file’s conventions (quoted, no '@'). Please confirm that "ankitm_me" is the intended Twitter handle for @ankitk-me.


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.

@thulieblack
Copy link
Member

/rtm

@asyncapi-bot asyncapi-bot merged commit d5a834b into asyncapi:master Oct 15, 2025
22 checks passed
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.

3 participants