-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
chore(blog): summary of the asyncapi mentorship program 2024 #4292
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
chore(blog): summary of the asyncapi mentorship program 2024 #4292
Conversation
✅ Deploy Preview for asyncapi-website ready!Built without sensitive environment variables
To edit notification comments on pull requests, go to your Netlify project configuration. |
WalkthroughA new blog post summarizing the 2024 AsyncAPI Mentorship Program has been added. The post reflects on the program's evolution, achievements, challenges, and future directions, and includes recognition for contributors. No changes were made to existing code or public entities. Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Suggested labels
Suggested reviewers
Poem
Note ⚡️ Unit Test Generation is now available in beta!Learn more here, or try it out under "Finishing Touches" below. ✨ Finishing Touches🧪 Generate unit tests
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. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. 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
Documentation and Community
|
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #4292 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 22 22
Lines 778 778
Branches 144 144
=========================================
Hits 778 778 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
⚡️ Lighthouse report for the changes in this PR:
Lighthouse ran on https://deploy-preview-4292--asyncapi-website.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.
Actionable comments posted: 1
🧹 Nitpick comments (2)
markdown/blog/2024-mentorship-program-summary.md (2)
159-159: Tighten wording & drop “truly amazing” clichéConsider a crisper sentence:
-And to our inspiring contributors: you journeyed through trials and triumphs, not only concluding the program with stellar results but also crafting truly amazing work. +To our inspiring contributors: you navigated trials and triumphs, finished with stellar results, and produced outstanding work.
14-14: Remove trailing whitespace in YAML front-matterLine 14 ends with an extra space after “Preacher”. This shows up in git-diff noise and some linters warn.
- byline: AsyncAPI Preacher␠ + byline: AsyncAPI Preacher
📜 Review details
Configuration used: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
markdown/blog/2024-mentorship-program-summary.md(1 hunks)
🧰 Additional context used
🧠 Learnings (2)
📓 Common learnings
Learnt from: asyncapi-bot
PR: asyncapi/website#0
File: :0-0
Timestamp: 2025-02-18T12:07:42.211Z
Learning: The following PR commands are supported in the asyncapi/website repository:
- `/please-take-a-look` or `/ptal`: Requests attention from reviewers who haven't reviewed the PR
- `/ready-to-merge` or `/rtm`: Triggers automerge when all conditions are met
- `/do-not-merge` or `/dnm`: Blocks automerge even if all conditions are met
- `/autoupdate` or `/au`: Adds autoupdate label to keep PR in sync with target branch
- `/update` or `/u`: One-time update of PR with latest changes from target branch
Learnt from: iambami
PR: asyncapi/website#4111
File: markdown/blog/community-spotlight-aayush.md:3-3
Timestamp: 2025-06-03T06:48:09.304Z
Learning: For AsyncAPI community spotlight blog posts, the frontmatter date should reflect the actual publication date, not the month being highlighted in the spotlight. For example, a "May Community Spotlight" can be published in June with a June date in the frontmatter.
markdown/blog/2024-mentorship-program-summary.md (1)
Learnt from: iambami
PR: #4111
File: markdown/blog/community-spotlight-aayush.md:3-3
Timestamp: 2025-06-03T06:48:09.304Z
Learning: For AsyncAPI community spotlight blog posts, the frontmatter date should reflect the actual publication date, not the month being highlighted in the spotlight. For example, a "May Community Spotlight" can be published in June with a June date in the frontmatter.
🪛 LanguageTool
markdown/blog/2024-mentorship-program-summary.md
[style] ~159-~159: Opting for a less wordy alternative here may improve the clarity of your writing.
Context: ... journeyed through trials and triumphs, not only concluding the program with stellar results but also crafting truly amazing work. Your dedication lig...
(NOT_ONLY_ALSO)
[style] ~159-~159: Consider using a more formal and expressive alternative to ‘amazing’.
Context: ...stellar results but also crafting truly amazing work. Your dedication lights the way fo...
(AWESOME)
🪛 markdownlint-cli2 (0.17.2)
markdown/blog/2024-mentorship-program-summary.md
202-202: Bare URL used
(MD034, no-bare-urls)
203-203: Blank line inside blockquote
(MD028, no-blanks-blockquote)
204-204: Bare URL used
(MD034, no-bare-urls)
⏰ 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: Test NodeJS PR - macos-13
- GitHub Check: Test NodeJS PR - windows-latest
- GitHub Check: Lighthouse CI
🔇 Additional comments (1)
markdown/blog/2024-mentorship-program-summary.md (1)
3-3: Confirm publication date is correctFront-matter date is set to
2025-08-01. Double-check this aligns with the actual planned publish date so that the post appears in the right chronological order on the site.
| > Photo by Andrea Piacquadio: https://www.pexels.com/photo/amazed-formal-male-looking-at-laptop-screen-3760809/ | ||
|
|
||
| > Photo from independent.co.uk : https://www.independent.co.uk/life-style/health-and-families/tired-fatigue-symptoms-reason-help-tips-b2669575.html No newline at end of file |
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.
Fix markdown-lint errors: remove blank line inside blockquote & replace bare URLs with links
markdownlint flags MD028 (blank line) and MD034 (bare URLs). Both will fail CI. Patch:
-> Photo by Andrea Piacquadio: https://www.pexels.com/photo/amazed-formal-male-looking-at-laptop-screen-3760809/
-
-> Photo from independent.co.uk : https://www.independent.co.uk/life-style/health-and-families/tired-fatigue-symptoms-reason-help-tips-b2669575.html
+> Photo by [Andrea Piacquadio](https://www.pexels.com/photo/amazed-formal-male-looking-at-laptop-screen-3760809/)
+> Photo from independent.co.uk: [Tiredness article](https://www.independent.co.uk/life-style/health-and-families/tired-fatigue-symptoms-reason-help-tips-b2669575.html)📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| > Photo by Andrea Piacquadio: https://www.pexels.com/photo/amazed-formal-male-looking-at-laptop-screen-3760809/ | |
| > Photo from independent.co.uk : https://www.independent.co.uk/life-style/health-and-families/tired-fatigue-symptoms-reason-help-tips-b2669575.html | |
| > Photo by [Andrea Piacquadio](https://www.pexels.com/photo/amazed-formal-male-looking-at-laptop-screen-3760809/) | |
| > Photo from independent.co.uk: [Tiredness article](https://www.independent.co.uk/life-style/health-and-families/tired-fatigue-symptoms-reason-help-tips-b2669575.html) |
🧰 Tools
🪛 markdownlint-cli2 (0.17.2)
202-202: Bare URL used
(MD034, no-bare-urls)
203-203: Blank line inside blockquote
(MD028, no-blanks-blockquote)
204-204: Bare URL used
(MD034, no-bare-urls)
🤖 Prompt for AI Agents
In markdown/blog/2024-mentorship-program-summary.md around lines 202 to 204,
remove the blank line inside the blockquote to fix the MD028 markdownlint error
and replace the bare URLs with proper markdown links to fix the MD034 error.
Format the URLs as clickable links with descriptive text instead of plain URLs.
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: 1
🧹 Nitpick comments (1)
markdown/blog/2024-mentorship-program-summary.md (1)
159-159: Wording nitpickLanguageTool flags “truly amazing work.” Consider a crisper alternative such as “exceptional work” to keep the tone professional.
📜 Review details
Configuration used: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
markdown/blog/2024-mentorship-program-summary.md(1 hunks)
🧰 Additional context used
🧠 Learnings (2)
📓 Common learnings
Learnt from: asyncapi-bot
PR: asyncapi/website#0
File: :0-0
Timestamp: 2025-02-18T12:07:42.211Z
Learning: The following PR commands are supported in the asyncapi/website repository:
- `/please-take-a-look` or `/ptal`: Requests attention from reviewers who haven't reviewed the PR
- `/ready-to-merge` or `/rtm`: Triggers automerge when all conditions are met
- `/do-not-merge` or `/dnm`: Blocks automerge even if all conditions are met
- `/autoupdate` or `/au`: Adds autoupdate label to keep PR in sync with target branch
- `/update` or `/u`: One-time update of PR with latest changes from target branch
Learnt from: akshatnema
PR: asyncapi/website#3265
File: tests/fixtures/toolsObjectData.js:51-52
Timestamp: 2024-10-09T17:35:36.557Z
Learning: When reviewing code in the 'asyncapi/website' repository, akshatnema prefers that I do not provide committable code suggestions.
Learnt from: iambami
PR: asyncapi/website#4111
File: markdown/blog/community-spotlight-aayush.md:3-3
Timestamp: 2025-06-03T06:48:09.304Z
Learning: For AsyncAPI community spotlight blog posts, the frontmatter date should reflect the actual publication date, not the month being highlighted in the spotlight. For example, a "May Community Spotlight" can be published in June with a June date in the frontmatter.
markdown/blog/2024-mentorship-program-summary.md (3)
Learnt from: iambami
PR: #4111
File: markdown/blog/community-spotlight-aayush.md:3-3
Timestamp: 2025-06-03T06:48:09.304Z
Learning: For AsyncAPI community spotlight blog posts, the frontmatter date should reflect the actual publication date, not the month being highlighted in the spotlight. For example, a "May Community Spotlight" can be published in June with a June date in the frontmatter.
Learnt from: akshatnema
PR: #3378
File: scripts/markdown/check-markdown.js:1-1
Timestamp: 2024-11-25T18:34:51.303Z
Learning: When reviewing scripts/markdown/check-markdown.js, optimizations should be addressed in separate issues and not included in the current pull request.
Learnt from: akshatnema
PR: #3262
File: components/navigation/BlogPostItem.tsx:95-119
Timestamp: 2024-10-11T11:32:30.226Z
Learning: In the BlogPostItem component (components/navigation/BlogPostItem.tsx), nesting <a> tags inside the parent Link component leads to hydration issues; therefore, we should avoid nesting <a> tags inside Link components in this component.
🪛 LanguageTool
markdown/blog/2024-mentorship-program-summary.md
[style] ~159-~159: Opting for a less wordy alternative here may improve the clarity of your writing.
Context: ... journeyed through trials and triumphs, not only concluding the program with stellar results but also crafting truly amazing work. Your dedication lig...
(NOT_ONLY_ALSO)
[style] ~159-~159: Consider using a more formal and expressive alternative to ‘amazing’.
Context: ...stellar results but also crafting truly amazing work. Your dedication lights the way fo...
(AWESOME)
🪛 markdownlint-cli2 (0.17.2)
markdown/blog/2024-mentorship-program-summary.md
203-203: Bare URL used
(MD034, no-bare-urls)
204-204: Bare URL used
(MD034, no-bare-urls)
⏰ 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). (1)
- GitHub Check: Test NodeJS PR - windows-latest
| > Photo by Andrea Piacquadio: https://www.pexels.com/photo/amazed-formal-male-looking-at-laptop-screen-3760809/ | ||
| > Photo from independent.co.uk : https://www.independent.co.uk/life-style/health-and-families/tired-fatigue-symptoms-reason-help-tips-b2669575.html No newline at end of file |
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.
Fix MD034: replace bare URLs with markdown links
markdownlint fails on bare URLs here (MD034). Convert each to a descriptive markdown link, e.g.:
> Photo by [Andrea Piacquadio](https://www.pexels.com/photo/amazed-formal-male-looking-at-laptop-screen-3760809/)
Do the same for the second citation to unblock CI.
🧰 Tools
🪛 markdownlint-cli2 (0.17.2)
203-203: Bare URL used
(MD034, no-bare-urls)
204-204: Bare URL used
(MD034, no-bare-urls)
🤖 Prompt for AI Agents
In markdown/blog/2024-mentorship-program-summary.md around lines 203 to 204, the
bare URLs violate markdownlint rule MD034. Replace each bare URL with a markdown
link by wrapping the link text in brackets and the URL in parentheses, for
example, change the first line to "Photo by [Andrea
Piacquadio](https://www.pexels.com/photo/amazed-formal-male-looking-at-laptop-screen-3760809/)"
and similarly convert the second citation to a markdown link format.
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.
This is something to consider @AceTheCreator.
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.
Seems like the humans are having a chat. I'll hop back into my burrow for now. If you need me again, just tag @coderabbitai in a new comment, and I'll come hopping out!
bandantonio
left a comment
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.
@AceTheCreator Aside from two minor comments, this is an amazing work! Thank you!
|
|
||
| The 2024 cohort brought to light that even our highly capable volunteer mentors, especially those new to the program, aren't always immediately proficient in every aspect of mentorship. We noted with surprise that some initially struggled with critical tasks like defining project goals, setting milestones, and establishing communication frequencies. Without these clear frameworks, evaluating both technical progress and behavioral growth became challenging for us. | ||
|
|
||
| Another unexpected development was the unfortunate reality of mentor withdrawals. While we typically anticipate and prepare for mentees dropping out due to personal reasons, a mentor's early departure was unprecedented for us. |
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.
[suggestion]: It would be helpful to mention the mentor that ended up leaving and how it impacted the cohort. That way readers can understand why their departure was a shock.
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.
I don't think it's necessary to mention who the original mentors were, as they stepped away due to personal reasons that I won’t be disclosing in this article. Their departure didn’t affect the program's success, as we had incredible mentors step in and seamlessly carry things forward.
CBID2
left a comment
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.
Hi @AceTheCreator! :) The blog post is of to a great start! :) I pointed out some typos and added some questions that can help you improve it even more. Keep it up! 👍🏾
|
@CBID2 Your suggestions are way too formal. Remember, this is a blog post, not an airplane's documentation or instructions. The main goal of a blog post is not just to inform, but also connect with the community, so the writing style should be more casual, friendly, vibrant, and engaging. Each of our community members has their unique writing style and that's great! This is what makes each blog post special even if there is some informality or imperfect word choice. @AceTheCreator FYI ^^ Please keep the post engaging. I would rather fix grammar-related things, like some of the articles and links and left everything else as is. |
Noted @bandantonio. I'll go through my comments tomorrow and find a way to make them more grammar-focused and how they can be more engaging😊 |
Co-authored-by: Antonio <bandantonio@users.noreply.github.com>
Co-authored-by: Antonio <bandantonio@users.noreply.github.com>
thulieblack
left a comment
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.
minor changes
Co-authored-by: V Thulisile Sibanda <66913810+thulieblack@users.noreply.github.com>
Co-authored-by: V Thulisile Sibanda <66913810+thulieblack@users.noreply.github.com>
Co-authored-by: V Thulisile Sibanda <66913810+thulieblack@users.noreply.github.com>
Co-authored-by: V Thulisile Sibanda <66913810+thulieblack@users.noreply.github.com>
Co-authored-by: V Thulisile Sibanda <66913810+thulieblack@users.noreply.github.com>
Co-authored-by: V Thulisile Sibanda <66913810+thulieblack@users.noreply.github.com>
Co-authored-by: V Thulisile Sibanda <66913810+thulieblack@users.noreply.github.com>
Co-authored-by: V Thulisile Sibanda <66913810+thulieblack@users.noreply.github.com>
|
Commited your suggested changes @thulieblack 👍🏾 |
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
♻️ Duplicate comments (1)
markdown/blog/2024-mentorship-program-summary.md (1)
203-204: Bare URLs still violate markdownlint MD034
markdownlintwill keep failing until the two bare URLs inside the block-quote are converted to proper markdown links.
Replace each plain URL with the[text](url)format (e.g.,Photo by [Andrea Piacquadio](https://… )).
🧹 Nitpick comments (1)
markdown/blog/2024-mentorship-program-summary.md (1)
34-36: Typo: use “GitHub” instead of “Github”The brand is capital-H, so the anchor text should read “GitHub Discussion”.
📜 Review details
Configuration used: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
markdown/blog/2024-mentorship-program-summary.md(1 hunks)
🧰 Additional context used
🧠 Learnings (5)
📓 Common learnings
Learnt from: asyncapi-bot
PR: asyncapi/website#0
File: :0-0
Timestamp: 2025-02-18T12:07:42.211Z
Learning: The following PR commands are supported in the asyncapi/website repository:
- `/please-take-a-look` or `/ptal`: Requests attention from reviewers who haven't reviewed the PR
- `/ready-to-merge` or `/rtm`: Triggers automerge when all conditions are met
- `/do-not-merge` or `/dnm`: Blocks automerge even if all conditions are met
- `/autoupdate` or `/au`: Adds autoupdate label to keep PR in sync with target branch
- `/update` or `/u`: One-time update of PR with latest changes from target branch
Learnt from: akshatnema
PR: asyncapi/website#3265
File: tests/fixtures/toolsObjectData.js:51-52
Timestamp: 2024-10-09T17:35:36.557Z
Learning: When reviewing code in the 'asyncapi/website' repository, akshatnema prefers that I do not provide committable code suggestions.
Learnt from: iambami
PR: asyncapi/website#4111
File: markdown/blog/community-spotlight-aayush.md:3-3
Timestamp: 2025-06-03T06:48:09.304Z
Learning: For AsyncAPI community spotlight blog posts, the frontmatter date should reflect the actual publication date, not the month being highlighted in the spotlight. For example, a "May Community Spotlight" can be published in June with a June date in the frontmatter.
📚 Learning: for asyncapi community spotlight blog posts, the frontmatter date should reflect the actual publicat...
Learnt from: iambami
PR: asyncapi/website#4111
File: markdown/blog/community-spotlight-aayush.md:3-3
Timestamp: 2025-06-03T06:48:09.304Z
Learning: For AsyncAPI community spotlight blog posts, the frontmatter date should reflect the actual publication date, not the month being highlighted in the spotlight. For example, a "May Community Spotlight" can be published in June with a June date in the frontmatter.
Applied to files:
markdown/blog/2024-mentorship-program-summary.md
📚 Learning: when reviewing `scripts/markdown/check-markdown.js`, optimizations should be addressed in separate i...
Learnt from: akshatnema
PR: asyncapi/website#3378
File: scripts/markdown/check-markdown.js:1-1
Timestamp: 2024-11-25T18:34:51.303Z
Learning: When reviewing `scripts/markdown/check-markdown.js`, optimizations should be addressed in separate issues and not included in the current pull request.
Applied to files:
markdown/blog/2024-mentorship-program-summary.md
📚 Learning: in the `blogpostitem` component (`components/navigation/blogpostitem.tsx`), nesting `` tags insid...
Learnt from: akshatnema
PR: asyncapi/website#3262
File: components/navigation/BlogPostItem.tsx:95-119
Timestamp: 2024-10-11T11:32:30.226Z
Learning: In the `BlogPostItem` component (`components/navigation/BlogPostItem.tsx`), nesting `<a>` tags inside the parent `Link` component leads to hydration issues; therefore, we should avoid nesting `<a>` tags inside `Link` components in this component.
Applied to files:
markdown/blog/2024-mentorship-program-summary.md
📚 Learning: in the markdown files for the asyncapi website, avatar image paths should be referenced without the ...
Learnt from: bandantonio
PR: asyncapi/website#3393
File: markdown/blog/2024-Q1-docs-report.md:10-12
Timestamp: 2024-11-13T20:34:18.998Z
Learning: In the markdown files for the AsyncAPI website, avatar image paths should be referenced without the `public` prefix (e.g., use `/img/avatars/...` instead of `/public/img/avatars/...`).
Applied to files:
markdown/blog/2024-mentorship-program-summary.md
🪛 LanguageTool
markdown/blog/2024-mentorship-program-summary.md
[style] ~159-~159: Opting for a less wordy alternative here may improve the clarity of your writing.
Context: ... journeyed through trials and triumphs, not only concluding the program with stellar results but also crafting truly amazing work. Your dedication lig...
(NOT_ONLY_ALSO)
[style] ~159-~159: Consider using a more formal and expressive alternative to ‘amazing’.
Context: ...stellar results but also crafting truly amazing work. Your dedication lights the way fo...
(AWESOME)
🪛 markdownlint-cli2 (0.17.2)
markdown/blog/2024-mentorship-program-summary.md
203-203: Bare URL used
(MD034, no-bare-urls)
204-204: Bare URL used
(MD034, no-bare-urls)
⏰ 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). (2)
- GitHub Check: Test NodeJS PR - macos-13
- GitHub Check: Test NodeJS PR - windows-latest
🔇 Additional comments (1)
markdown/blog/2024-mentorship-program-summary.md (1)
39-44: Verify that images rendered with<Figure>have meaningful alt textThe custom
<Figure>component receivessrcandcaption, but no explicitalt.
Please ensure the component internally sets analtattribute (or uses the caption) so screen-reader users get descriptive text.
thulieblack
left a comment
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.
✅
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
♻️ Duplicate comments (1)
markdown/blog/2024-mentorship-program-summary.md (1)
203-204: Bare URLs still violate markdown-lint rules (MD034)
This issue has been flagged in earlier review comments and will fail CI. Replace each raw URL with a markdown link.
🧹 Nitpick comments (2)
markdown/blog/2024-mentorship-program-summary.md (2)
1-4: Verify the front-matter date reflects the actual planned publication date
Previous reviews on the website repo emphasized that thedatefield should correspond to when the post goes live, not necessarily the cohort year being discussed. Double-check that2025-08-01T06:00:00+01:00is indeed the intended publish date.
115-156: Consider serving mentor & contributor avatars from local assets
All avatar URLs point to GitHub’s CDN. Relying on external domains can slow page loads and break offline builds. Hosting copies under/img/avatars/…(as done elsewhere in the site) keeps assets under your control and avoids mixed-content issues if the site is ever served over different protocols.Also applies to: 161-197
📜 Review details
Configuration used: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
markdown/blog/2024-mentorship-program-summary.md(1 hunks)
🧰 Additional context used
🧠 Learnings (6)
📓 Common learnings
Learnt from: asyncapi-bot
PR: asyncapi/website#0
File: :0-0
Timestamp: 2025-02-18T12:07:42.211Z
Learning: The following PR commands are supported in the asyncapi/website repository:
- `/please-take-a-look` or `/ptal`: Requests attention from reviewers who haven't reviewed the PR
- `/ready-to-merge` or `/rtm`: Triggers automerge when all conditions are met
- `/do-not-merge` or `/dnm`: Blocks automerge even if all conditions are met
- `/autoupdate` or `/au`: Adds autoupdate label to keep PR in sync with target branch
- `/update` or `/u`: One-time update of PR with latest changes from target branch
Learnt from: akshatnema
PR: asyncapi/website#3265
File: tests/fixtures/toolsObjectData.js:51-52
Timestamp: 2024-10-09T17:35:36.557Z
Learning: When reviewing code in the 'asyncapi/website' repository, akshatnema prefers that I do not provide committable code suggestions.
Learnt from: iambami
PR: asyncapi/website#4111
File: markdown/blog/community-spotlight-aayush.md:3-3
Timestamp: 2025-06-03T06:48:09.304Z
Learning: For AsyncAPI community spotlight blog posts, the frontmatter date should reflect the actual publication date, not the month being highlighted in the spotlight. For example, a "May Community Spotlight" can be published in June with a June date in the frontmatter.
📚 Learning: for asyncapi community spotlight blog posts, the frontmatter date should reflect the actual publicat...
Learnt from: iambami
PR: asyncapi/website#4111
File: markdown/blog/community-spotlight-aayush.md:3-3
Timestamp: 2025-06-03T06:48:09.304Z
Learning: For AsyncAPI community spotlight blog posts, the frontmatter date should reflect the actual publication date, not the month being highlighted in the spotlight. For example, a "May Community Spotlight" can be published in June with a June date in the frontmatter.
Applied to files:
markdown/blog/2024-mentorship-program-summary.md
📚 Learning: when reviewing `scripts/markdown/check-markdown.js`, optimizations should be addressed in separate i...
Learnt from: akshatnema
PR: asyncapi/website#3378
File: scripts/markdown/check-markdown.js:1-1
Timestamp: 2024-11-25T18:34:51.303Z
Learning: When reviewing `scripts/markdown/check-markdown.js`, optimizations should be addressed in separate issues and not included in the current pull request.
Applied to files:
markdown/blog/2024-mentorship-program-summary.md
📚 Learning: in the `blogpostitem` component (`components/navigation/blogpostitem.tsx`), nesting `` tags insid...
Learnt from: akshatnema
PR: asyncapi/website#3262
File: components/navigation/BlogPostItem.tsx:95-119
Timestamp: 2024-10-11T11:32:30.226Z
Learning: In the `BlogPostItem` component (`components/navigation/BlogPostItem.tsx`), nesting `<a>` tags inside the parent `Link` component leads to hydration issues; therefore, we should avoid nesting `<a>` tags inside `Link` components in this component.
Applied to files:
markdown/blog/2024-mentorship-program-summary.md
📚 Learning: in the markdown files for the asyncapi website, avatar image paths should be referenced without the ...
Learnt from: bandantonio
PR: asyncapi/website#3393
File: markdown/blog/2024-Q1-docs-report.md:10-12
Timestamp: 2024-11-13T20:34:18.998Z
Learning: In the markdown files for the AsyncAPI website, avatar image paths should be referenced without the `public` prefix (e.g., use `/img/avatars/...` instead of `/public/img/avatars/...`).
Applied to files:
markdown/blog/2024-mentorship-program-summary.md
📚 Learning: the asyncapi website modularization pr uses promise.allsettled for parallel execution of build tasks...
Learnt from: sagarkori143
PR: asyncapi/website#0
File: :0-0
Timestamp: 2025-06-20T14:47:22.389Z
Learning: The AsyncAPI website modularization PR uses Promise.allSettled for parallel execution of build tasks, which provides better performance and fault isolation compared to sequential execution.
Applied to files:
markdown/blog/2024-mentorship-program-summary.md
🪛 LanguageTool
markdown/blog/2024-mentorship-program-summary.md
[style] ~159-~159: Opting for a less wordy alternative here may improve the clarity of your writing.
Context: ... journeyed through trials and triumphs, not only concluding the program with stellar results but also crafting truly amazing work. Your dedication lig...
(NOT_ONLY_ALSO)
[style] ~159-~159: Consider using a more formal and expressive alternative to ‘amazing’.
Context: ...stellar results but also crafting truly amazing work. Your dedication lights the way fo...
(AWESOME)
🪛 markdownlint-cli2 (0.17.2)
markdown/blog/2024-mentorship-program-summary.md
203-203: Bare URL used
(MD034, no-bare-urls)
204-204: Bare URL used
(MD034, no-bare-urls)
🔇 Additional comments (1)
markdown/blog/2024-mentorship-program-summary.md (1)
39-44: Figure component missing explicit alt text
For accessibility,<Figure>should convey meaningful alternative text. Thecaptionhelps but many screen-readers still rely on analtattribute. Confirm that the component internally mapscaptiontoalt; if not, add analtprop.
sambhavgupta0705
left a comment
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
|
/rtm |
CBID2
left a comment
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! 😊
|
/rtm |
This PR is a summary article for the 2024 AsyncAPI Mentorship Program. It captures the journey, achievements, challenges, and future strategic direction of the program, setting the stage for the upcoming "AsyncAPI Maintainership Program.
Summary by CodeRabbit