Skip to content

Conversation

@asyncapi-bot
Copy link
Contributor

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

Automated changes by create-pull-request GitHub action

Summary by CodeRabbit

  • Chores
    • Removed the "AsyncAPI Design WG Meeting" from the meetings list, including its calendar link, URL, banner, and date.
    • The meeting no longer appears in schedules or event listings across the app.
    • This update reflects the current meeting lineup and helps prevent confusion by removing an outdated entry.

@netlify
Copy link

netlify bot commented Oct 12, 2025

Deploy Preview for asyncapi-website ready!

Name Link
🔨 Latest commit 136354b
🔍 Latest deploy log https://app.netlify.com/projects/asyncapi-website/deploys/68eaf85d41591800089e6fc8
😎 Deploy Preview https://deploy-preview-4465--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 12, 2025

Walkthrough

Removed the "AsyncAPI Design WG Meeting" entry from config/meetings.json by deleting its meeting object (including calLink, url, banner, and date fields).

Changes

Cohort / File(s) Change Summary
Meetings config cleanup
config/meetings.json
Removed the "AsyncAPI Design WG Meeting" object, including its calLink, url, banner, and date fields.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

Suggested reviewers

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

Poem

Thump-thump, I hop through JSON fields,
A meeting gone—the config yields.
Carrots counted, arrays neat,
One less nibble in the list to eat.
With tidy paws, I prune and preen—
The schedule’s light, the meadow clean. 🥕🐇

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Title Check ⚠️ Warning The title lists file names and thus is overly generic and does not describe the actual content of the change. It also contains a typo in “newsrooom_videos.json,” which may confuse reviewers. The title fails to convey the removal of the AsyncAPI Design WG Meeting entry or the specific nature of updates to the video listings. A descriptive title should focus on the substantive changes rather than file paths. Consider renaming the PR to reflect the key updates rather than listing file names. For example: “chore: remove AsyncAPI Design WG Meeting entry from meetings.json and update newsroom videos listing” clearly communicates what was changed and avoids typos. This will make the change more understandable for reviewers and maintainers.
✅ 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/19e3c346952c7cb8401e4fd2822273e4e0666e74

📜 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 19e3c34 and 136354b.

📒 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: Lighthouse CI
  • GitHub Check: Test NodeJS PR - macos-13
  • GitHub Check: Test NodeJS PR - windows-latest

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 12, 2025

Codecov Report

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

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

@asyncapi-bot asyncapi-bot merged commit 4fd7abd into master Oct 12, 2025
37 of 38 checks passed
@asyncapi-bot asyncapi-bot deleted the update-meetings/19e3c346952c7cb8401e4fd2822273e4e0666e74 branch October 12, 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