Skip to content

Conversation

@asyncapi-bot
Copy link
Contributor

@asyncapi-bot asyncapi-bot commented Dec 31, 2025

Automated changes by create-pull-request GitHub action

Summary by CodeRabbit

  • Chores
    • Removed archived newsroom video entries to streamline the video library, keeping the collection focused on recent content.

✏️ Tip: You can customize this high-level summary in your review settings.

@netlify
Copy link

netlify bot commented Dec 31, 2025

Deploy Preview for asyncapi-website ready!

Name Link
🔨 Latest commit 6505603
🔍 Latest deploy log https://app.netlify.com/projects/asyncapi-website/deploys/695471dab101fe0008d643a5
😎 Deploy Preview https://deploy-preview-4816--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.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Dec 31, 2025

📝 Walkthrough

Walkthrough

A large block of video entries has been removed from the newsroom_videos.json configuration file. The array now terminates with the Governance Board Meeting entry dated November 3rd, 2025, with all subsequent historical video entries deleted.

Changes

Cohort / File(s) Change Summary
Configuration Data
config/newsroom_videos.json
Removed multiple video entry objects (containing image_url, title, description, and videoId fields) from the video array, reducing the dataset and shortening the historical record to end with the November 3rd, 2025 Governance Board Meeting entry

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

  • asyncapi/website#4806: Both PRs modify the config/newsroom_videos.json video array by removing/adding entries in the same structure
  • asyncapi/website#4695: Both PRs directly alter the video entry objects in the newsroom_videos.json array, with complementary removal and addition operations
  • asyncapi/website#4742: Both PRs modify the trailing entries in config/newsroom_videos.json's video array structure

Suggested reviewers

  • derberg
  • akshatnema
  • anshgoyalevil
  • sambhavgupta0705
  • Mayaleeeee
  • princerajpoot20

Poem

🐰 Hop through the config, trim the old away,
Videos past make room for today,
November's meeting now marks the end,
Cleaner archives—a rabbit's best friend! 🎬✨

Pre-merge checks

❌ Failed checks (1 inconclusive)
Check name Status Explanation Resolution
Title check ❓ Inconclusive The title mentions updating 'meetings.json and newsrooom_videos.json', but the raw summary shows only newsroom_videos.json was modified with deletions. The PR objectives indicate meetings.json and dashboard.json were also updated, yet the summary only details newsroom_videos.json changes. This creates a mismatch between what the title claims and what is documented. Verify which files were actually modified in this PR. If meetings.json and dashboard.json have meaningful changes, the title should accurately reflect them or be more general. If only newsroom_videos.json changed, the title should be corrected to reflect that.
✅ 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 changed files to evaluate docstring coverage. Skipping docstring coverage check.

📜 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 5b8a1c2 and 6505603.

📒 Files selected for processing (1)
  • config/newsroom_videos.json
💤 Files with no reviewable changes (1)
  • config/newsroom_videos.json
⏰ 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). (5)
  • GitHub Check: Automerge PR autoapproved by a bot
  • GitHub Check: cypress-run
  • GitHub Check: Test NodeJS PR - windows-latest
  • GitHub Check: Test NodeJS PR - macos-latest
  • GitHub Check: Lighthouse CI

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 Dec 31, 2025

Codecov Report

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

Additional details and impacted files
@@            Coverage Diff            @@
##            master     #4816   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           22        22           
  Lines          798       798           
  Branches       146       146           
=========================================
  Hits           798       798           

☔ 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 46
🟢 Accessibility 98
🟢 Best practices 92
🟢 SEO 100
🔴 PWA 33

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

@asyncapi-bot asyncapi-bot merged commit c1473dc into master Dec 31, 2025
38 checks passed
@asyncapi-bot asyncapi-bot deleted the update-meetings/5b8a1c23a16f2517a9db322ab18e84b5904d1acf branch December 31, 2025 00:48
SHUBHANSHU602 pushed a commit to SHUBHANSHU602/website that referenced this pull request Jan 2, 2026
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