Skip to content

Conversation

@asyncapi-bot
Copy link
Contributor

@asyncapi-bot asyncapi-bot commented Nov 15, 2025

Automated changes by create-pull-request GitHub action

Summary by CodeRabbit

  • New Features
    • Added "AsyncAPI Spec Triage" meeting scheduled for November 19, 2025 at 2:00 PM UTC.

@netlify
Copy link

netlify bot commented Nov 15, 2025

Deploy Preview for asyncapi-website ready!

Name Link
🔨 Latest commit 3809723
🔍 Latest deploy log https://app.netlify.com/projects/asyncapi-website/deploys/6917cb4d4cbb7a000855ba7b
😎 Deploy Preview https://deploy-preview-4590--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 Nov 15, 2025

Walkthrough

A new meeting entry titled "AsyncAPI Spec Triage" is added to the meetings configuration file, scheduled for November 19, 2025. The entry includes a Google Calendar link, GitHub issue reference, and metadata. No existing entries are modified.

Changes

Cohort / File(s) Summary
Meeting Configuration Update
config/meetings.json
Added new meeting entry for "AsyncAPI Spec Triage" scheduled for 2025-11-19T14:00:00.000Z with associated calendar and GitHub links

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

  • Verify the meeting date and calendar URL are correct
  • Confirm the GitHub issue link is properly formatted and accessible

Possibly related PRs

Suggested labels

autoapproved, autoupdate

Suggested reviewers

  • derberg
  • akshatnema
  • anshgoyalevil
  • sambhavgupta0705
  • Mayaleeeee

Poem

🐰 A gathering new in the calendar falls,
Async API devs answering the calls,
Triage meets spec on a November morn,
Another meeting entry is born! 📅✨

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Title check ⚠️ Warning The PR title mentions updating both meetings.json and newsroom_videos.json, but the changeset only contains modifications to config/meetings.json with no changes to newsroom_videos.json. Update the PR title to accurately reflect the actual changes: 'chore: add AsyncAPI Spec Triage meeting entry to meetings.json' or similar, removing the reference to newsroom_videos.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.
✨ 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/b87dd1bc1d103d58c774338daac016ecb490cf17

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 Nov 15, 2025

Codecov Report

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

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

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

Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 0

🧹 Nitpick comments (1)
config/meetings.json (1)

115-115: Remove trailing whitespace from title.

The title field contains a trailing space that should be removed: "AsyncAPI Spec Triage ""AsyncAPI Spec Triage". While line 17 has a similar issue, best practice is to eliminate trailing whitespace.

-    "title": "AsyncAPI Spec Triage ",
+    "title": "AsyncAPI Spec Triage",
📜 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 b87dd1b and 3809723.

📒 Files selected for processing (1)
  • config/meetings.json (1 hunks)
🧰 Additional context used
🧠 Learnings (1)
📓 Common learnings
Learnt from: asyncapi-bot
Repo: asyncapi/website PR: 0
File: :0-0
Timestamp: 2025-02-18T12:07:42.211Z
Learning: The following PR commands are supported in the asyncapi/website repository:
- `/please-take-a-look` or `/ptal`: Requests attention from reviewers who haven't reviewed the PR
- `/ready-to-merge` or `/rtm`: Triggers automerge when all conditions are met
- `/do-not-merge` or `/dnm`: Blocks automerge even if all conditions are met
- `/autoupdate` or `/au`: Adds autoupdate label to keep PR in sync with target branch
- `/update` or `/u`: One-time update of PR with latest changes from target branch
⏰ 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). (7)
  • GitHub Check: Automerge PR autoapproved by a bot
  • GitHub Check: Redirect rules - asyncapi-website
  • GitHub Check: Header rules - asyncapi-website
  • GitHub Check: Pages changed - asyncapi-website
  • GitHub Check: Lighthouse CI
  • GitHub Check: Test NodeJS PR - windows-latest
  • GitHub Check: Test NodeJS PR - macos-13
🔇 Additional comments (1)
config/meetings.json (1)

114-120: Verify this is the only file change in the PR.

The PR title mentions both meetings.json and newsroom_videos.json, but only meetings.json was provided for review. Please confirm whether newsroom_videos.json was intentionally excluded or if it should also be reviewed.

@asyncapi-bot
Copy link
Contributor Author

⚡️ Lighthouse report for the changes in this PR:

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

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

@asyncapi-bot asyncapi-bot merged commit 5bfca09 into master Nov 15, 2025
38 checks passed
@asyncapi-bot asyncapi-bot deleted the update-meetings/b87dd1bc1d103d58c774338daac016ecb490cf17 branch November 15, 2025 00:43
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