Skip to content

Conversation

@asyncapi-bot
Copy link
Contributor

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

Automated changes by create-pull-request GitHub action

Summary by CodeRabbit

  • New Features
    • Added a new newsroom video: “One Tool, One Flow: AsyncAPI’s New Take on Code/Docs/Config Generation,” complete with image, description, and playback (videoId: Mkd7FgKOMNE).
    • Removed the “AsyncAPI Mentorship Program Wrap-Up - 2024” video to keep the lineup current.
    • Users will see the updated video list in the newsroom; no other videos were changed.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Oct 1, 2025

Walkthrough

Updated config/newsroom_videos.json by adding one new video entry and removing one existing entry. No other files or entries were modified.

Changes

Cohort / File(s) Summary
Newsroom videos config
config/newsroom_videos.json
Added video "One Tool, One Flow: AsyncAPI's New Take on Code/Docs/Config Generation" (videoId: Mkd7FgKOMNE); removed "AsyncAPI Mentorship Program Wrap-Up - 2024" (videoId: XjckzYopxoA).

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

Suggested labels

autoapproved, autoupdate

Suggested reviewers

  • derberg
  • akshatnema
  • anshgoyalevil
  • sambhavgupta0705
  • Mayaleeeee
  • devilkiller-ag
  • vishvamsinh28
  • asyncapi-bot-eve

Poem

A hop and a flick through JSON fields,
I swap one carrot for a fresher yield.
New frames burrow in, old ones take flight,
Mkd7FgKOMNE shines bright.
Thump-thump—newsroom tidy and neat,
Another meadow of links complete. 🥕✨

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Title Check ⚠️ Warning The current title literally lists the filenames—including a misspelling of “newsroom_videos.json”—and does not succinctly summarize the nature of the update, which should focus on the change itself rather than raw file names. Please revise the title to correct the typo and provide a concise summary of the change’s intent, for example “chore: update meetings schedule and refresh newsroom video entry.”
✅ 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
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch update-meetings/d4a1175ce99e9565ea26cf6339a461f1b5ac6816

📜 Recent 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 d4a1175 and 5eb4a08.

📒 Files selected for processing (1)
  • config/newsroom_videos.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). (6)
  • GitHub Check: Redirect rules - asyncapi-website
  • GitHub Check: Header rules - asyncapi-website
  • GitHub Check: Pages changed - asyncapi-website
  • GitHub Check: Automerge PR autoapproved by a bot
  • GitHub Check: Test NodeJS PR - windows-latest
  • GitHub Check: Lighthouse CI
🔇 Additional comments (1)
config/newsroom_videos.json (1)

20-25: LGTM!

The new video entry is properly structured and follows the established pattern. The image URL correctly corresponds to the video ID, and the HTML entity encoding (') in the title is appropriate for JSON data.


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.

@netlify
Copy link

netlify bot commented Oct 1, 2025

Deploy Preview for asyncapi-website ready!

Name Link
🔨 Latest commit 5eb4a08
🔍 Latest deploy log https://app.netlify.com/projects/asyncapi-website/deploys/68dc78add26e0300081645e2
😎 Deploy Preview https://deploy-preview-4441--asyncapi-website.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@codecov
Copy link

codecov bot commented Oct 1, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.00%. Comparing base (d4a1175) to head (5eb4a08).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff            @@
##            master     #4441   +/-   ##
=========================================
  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.

@asyncapi-bot
Copy link
Contributor Author

⚡️ Lighthouse report for the changes in this PR:

Category Score
🔴 Performance 37
🟢 Accessibility 98
🟢 Best practices 92
🟢 SEO 100
🔴 PWA 33

Lighthouse ran on https://deploy-preview-4441--asyncapi-website.netlify.app/

@asyncapi-bot asyncapi-bot merged commit 4234dda into master Oct 1, 2025
38 checks passed
@asyncapi-bot asyncapi-bot deleted the update-meetings/d4a1175ce99e9565ea26cf6339a461f1b5ac6816 branch October 1, 2025 00:46
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