Skip to content

Conversation

@asyncapi-bot
Copy link
Contributor

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

Automated changes by create-pull-request GitHub action

Summary by CodeRabbit

  • Chores
    • Removed one AsyncAPI Community WG Meeting from the publicly displayed meetings list. The associated title, calendar link, URL, banner, and date are no longer shown.
    • Ensures the meetings view reflects the current set of active meetings; all other meeting entries remain unchanged.
    • No impact on functionality or navigation; users will simply see one fewer meeting in the list.

@netlify
Copy link

netlify bot commented Oct 10, 2025

Deploy Preview for asyncapi-website ready!

Name Link
🔨 Latest commit c3b171d
🔍 Latest deploy log https://app.netlify.com/projects/asyncapi-website/deploys/68e855711cc7d4000820d1b5
😎 Deploy Preview https://deploy-preview-4461--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 Oct 10, 2025

Walkthrough

One meeting entry for "AsyncAPI Community WG Meeting" was removed from config/meetings.json. All other meeting entries remain unchanged. No schema or structural changes.

Changes

Cohort / File(s) Summary of Changes
Meetings config update
config/meetings.json
Removed a single "AsyncAPI Community WG Meeting" object (title, calLink, url, banner, date). No other entries modified.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

Suggested reviewers

  • derberg
  • akshatnema
  • anshgoyalevil
  • sambhavgupta0705
  • Mayaleeeee
  • devilkiller-ag

Poem

I nudge the calendar, hop-hop—one less meet,
A quiet slot opens, tidy and neat.
JSON fields shuffled, no fuss, no fray,
Carrot-shaped cursor saves the day.
Thump of approval—onward we go! 🥕✨

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Title Check ⚠️ Warning The current title states updates to both meetings.json and newsrooom_videos.json but only config/meetings.json was modified. It also includes a typo in the filename newsrooom_videos.json which does not exist in the changes. This discrepancy can mislead reviewers about the scope of the PR. Please update the title to accurately reflect the single change, for example “chore: remove outdated AsyncAPI Community WG meeting entry from meetings.json,” and remove the reference to newsrooom_videos.json to avoid confusion.
✅ 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 (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch update-meetings/7affdb1fc4bb87280a9d2284e281c81a6165924a

📜 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 7affdb1 and c3b171d.

📒 Files selected for processing (1)
  • config/meetings.json (0 hunks)
💤 Files with no reviewable changes (1)
  • config/meetings.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). (4)
  • GitHub Check: Automerge PR autoapproved by a bot
  • GitHub Check: Test NodeJS PR - windows-latest
  • GitHub Check: Test NodeJS PR - macos-13
  • 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 Oct 10, 2025

Codecov Report

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

Additional details and impacted files
@@            Coverage Diff            @@
##            master     #4461   +/-   ##
=========================================
  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-4461--asyncapi-website.netlify.app/

@asyncapi-bot asyncapi-bot merged commit 4b2284c into master Oct 10, 2025
38 checks passed
@asyncapi-bot asyncapi-bot deleted the update-meetings/7affdb1fc4bb87280a9d2284e281c81a6165924a branch October 10, 2025 00:42
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