Skip to content

Comments

chore: update meetings.json and newsrooom_videos.json#5060

Merged
asyncapi-bot merged 1 commit intomasterfrom
update-meetings/eb10b10dc4aea67f2bae85a01b71ed0da8dc5dee
Feb 2, 2026
Merged

chore: update meetings.json and newsrooom_videos.json#5060
asyncapi-bot merged 1 commit intomasterfrom
update-meetings/eb10b10dc4aea67f2bae85a01b71ed0da8dc5dee

Conversation

@asyncapi-bot
Copy link
Contributor

@asyncapi-bot asyncapi-bot commented Feb 2, 2026

Automated changes by create-pull-request GitHub action

Summary by CodeRabbit

  • Chores
    • Removed archived newsroom video entries.

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

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 2, 2026

📝 Walkthrough

Walkthrough

This PR removes a block of 120 lines containing archived newsroom video entries from the configuration file, retaining only the most recent video entry dated November 19th 2025.

Changes

Cohort / File(s) Summary
Newsroom Videos Configuration
config/newsroom_videos.json
Removed 120 lines containing multiple archived video entries with metadata (image_url, title, description, videoId), keeping only the most recent entry.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Possibly related PRs

Suggested labels

autoapproved, autoupdate

Suggested reviewers

  • derberg
  • anshgoyalevil
  • akshatnema
  • sambhavgupta0705
  • princerajpoot20

Poem

🐰 Old videos fade, like autumn leaves,

We prune the past with gentle ease,

November shines, fresh and new,

Config cleaned for me and you! ✨

🚥 Pre-merge checks | ✅ 2 | ❌ 1
❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Title check ⚠️ Warning The PR title mentions updating 'meetings.json and newsrooom_videos.json', but the actual changes only modify 'config/newsroom_videos.json' (removes 120 lines of video entries). The title is misleading about the scope of changes. Update the PR title to accurately reflect the actual changes, such as 'chore: remove outdated newsroom video entries' or 'chore: clean up newsroom_videos.json'. Verify no changes were made to meetings.json.
✅ 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.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch update-meetings/eb10b10dc4aea67f2bae85a01b71ed0da8dc5dee

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 Feb 2, 2026

Codecov Report

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

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

☔ 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.

@netlify
Copy link

netlify bot commented Feb 2, 2026

Deploy Preview for asyncapi-website ready!

Name Link
🔨 Latest commit 7429ed9
🔍 Latest deploy log https://app.netlify.com/projects/asyncapi-website/deploys/697ff54ffb98ed000827854e
😎 Deploy Preview https://deploy-preview-5060--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.

@asyncapi-bot asyncapi-bot merged commit 229d0b9 into master Feb 2, 2026
37 of 38 checks passed
@github-project-automation github-project-automation bot moved this from To Be Triaged to Done in Website - Kanban Feb 2, 2026
@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-5060--asyncapi-website.netlify.app/

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

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants