Skip to content

Conversation

@asyncapi-bot
Copy link
Contributor

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

Automated changes by create-pull-request GitHub action

Summary by CodeRabbit

  • Style

    • Minor whitespace adjustment in a configuration file for consistency; no content, entries, or structure were changed. No user-visible impact.
  • Chores

    • Non-functional formatting cleanup to keep config files consistent. No changes to behavior or interfaces; no action required by users or admins.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Sep 10, 2025

Caution

Review failed

The pull request is closed.

Walkthrough

A single whitespace-only edit: config/meetings.json now ends with a trailing newline after the final closing bracket. No JSON keys, values, or structure were added, removed, or changed.

Changes

Cohort / File(s) Summary of Changes
Formatting / Whitespace
config/meetings.json
Added a trailing newline after the final closing bracket; no semantic changes to data or structure.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

Suggested reviewers

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

Poem

I’m a small rabbit, light and spry,
I nudged a newline, blinked my eye.
No keys disturbed, the structure’s fine—
Just one soft hop at JSON’s line. 🐇


📜 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 2286f15 and b7249f7.

📒 Files selected for processing (1)
  • config/meetings.json (1 hunks)
✨ Finishing touches
🧪 Generate unit tests
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch update-meetings/1991a70fc27690c3deeb27dfb975fea908e9742e

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.

@netlify
Copy link

netlify bot commented Sep 10, 2025

Deploy Preview for asyncapi-website ready!

Name Link
🔨 Latest commit b7249f7
🔍 Latest deploy log https://app.netlify.com/projects/asyncapi-website/deploys/68c319421be68300081aba40
😎 Deploy Preview https://deploy-preview-4405--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.

@codecov
Copy link

codecov bot commented Sep 10, 2025

Codecov Report

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

Additional details and impacted files
@@            Coverage Diff            @@
##            master     #4405   +/-   ##
=========================================
  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

asyncapi-bot commented Sep 10, 2025

⚡️ 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-4405--asyncapi-website.netlify.app/

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 (3)
config/meetings.json (1)

127-133: LGTM. Please also keep list ordering predictable.

Addition looks consistent with existing schema. Consider keeping this array chronologically sorted (or confirm the UI sorts at render) so upcoming items don’t appear out of order.

config/newsroom_videos.json (2)

2-7: Fix minor text nits: stray space before comma + trailing period in URL.

Tighten punctuation to avoid visual glitches and accidental link breakage.

Apply this diff:

-    "title": "AsyncAPI Community WG Meeting , Tuesday September 9th 2025",
-    "description": "https://github.com/asyncapi/community/issues/2060.",
+    "title": "AsyncAPI Community WG Meeting, Tuesday September 9th 2025",
+    "description": "https://github.com/asyncapi/community/issues/2060"

2-7: Optional: image perf hygiene for better LCP.

If the renderer uses these thumbnails directly, ensure fixed dimensions and lazy-loading are applied in the UI. Also consider serving a smaller YouTube thumbnail (e.g., mqdefault) if quality suffices.

📜 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 1991a70 and e7b86be.

📒 Files selected for processing (2)
  • config/meetings.json (1 hunks)
  • config/newsroom_videos.json (1 hunks)

Copy link
Member

@sambhavgupta0705 sambhavgupta0705 left a comment

Choose a reason for hiding this comment

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

lgtm

@sambhavgupta0705 sambhavgupta0705 merged commit 232ab4d into master Sep 11, 2025
19 of 20 checks passed
@sambhavgupta0705 sambhavgupta0705 deleted the update-meetings/1991a70fc27690c3deeb27dfb975fea908e9742e branch September 11, 2025 18:53
sambhavgupta0705 added a commit that referenced this pull request Sep 18, 2025
* docs(community): update latest community documentation (#4407)

* chore: update meetings.json and newsrooom_videos.json (#4410)

* chore: update meetings.json and newsrooom_videos.json (#4405)

* chore: update meetings.json, newsrooom_videos.json and dashboard.json

* Update meetings.json

---------

Co-authored-by: asyncapi-bot <info@asyncapi.io>
Co-authored-by: Eve <bot+eve@asyncapi.io>
Co-authored-by: Sambhav Gupta <81870866+sambhavgupta0705@users.noreply.github.com>

* chore: update meetings.json and newsrooom_videos.json (#4411)

* fix: #4065 Made Case Studies page table section mobile responsive (#4403)

* fixes #4065. Made Case Studie page mobile responsive

* fixes #4065. Made Case Studie page mobile responsive

* fix: #4381

---------

Co-authored-by: Sambhav Gupta <81870866+sambhavgupta0705@users.noreply.github.com>

* chore(deps): bump axios from 1.8.2 to 1.12.1 (#4413)

* chore(deps): bump @babel/runtime and next-language-detector (#4414)

* chore: update tools.json (#4417)

* chore: update meetings.json and newsrooom_videos.json (#4419)

* ci: update of files from global .github repo (#4418)

* fix: broken AsyncAPI contributing guidelines link (#4395) (#4398)


Co-authored-by: SanidhyaMadheshia <sanidhyamadheshia@gmail.com>
Co-authored-by: Sambhav Gupta <81870866+sambhavgupta0705@users.noreply.github.com>
Co-authored-by: V Thulisile Sibanda <66913810+thulieblack@users.noreply.github.com>

---------

Co-authored-by: Chan <bot+chan@asyncapi.io>
Co-authored-by: asyncapi-bot <info@asyncapi.io>
Co-authored-by: Eve <bot+eve@asyncapi.io>
Co-authored-by: namanjain24-sudo <namanjainpy@gmail.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: SanidhyaMadheshia <139427620+SanidhyaMadheshia@users.noreply.github.com>
Co-authored-by: SanidhyaMadheshia <sanidhyamadheshia@gmail.com>
Co-authored-by: V Thulisile Sibanda <66913810+thulieblack@users.noreply.github.com>
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.

4 participants