Skip to content

Conversation

@me-pradeep
Copy link

@me-pradeep me-pradeep commented Dec 25, 2025

Closes #4774

This PR adds content to the "Where to Contribute" page in the
community onboarding section to help new contributors understand
different ways they can contribute to the AsyncAPI project.

{56AD5200-66A5-456F-87FB-69780CF1A573}

Summary by CodeRabbit

  • Documentation
    • Replaced the placeholder onboarding page with a comprehensive "Where to Contribute" guide. Adds detailed sections for Documentation, Code, Testing & Quality, Design & UX, and Community & Support, with example contribution ideas, pointers to relevant repositories, and clear getting-started guidance. Sections are organized for easy navigation.

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

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Dec 25, 2025

📝 Walkthrough

Walkthrough

Replaces the placeholder "under construction" page with a comprehensive "Where to Contribute" onboarding guide covering Documentation, Code, Testing and Quality, Design and User Experience, Community and Support, and How to Get Started.

Changes

Cohort / File(s) Change Summary
Community Onboarding Documentation
markdown/docs/community/000-onboarding/where-to-contribute.md
Replaced placeholder content with a full contributor onboarding guide. Added sections: Documentation, Code, Testing and Quality, Design and User Experience, Community and Support, and How to Get Started, including example contribution types and relevant repositories.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐇 A blank page turned to paths well known,
Docs and code and tests now shown,
Design and help for those who roam,
Step in, contribute, make this home,
A rabbit cheers — you’re not alone!

Pre-merge checks and finishing touches

✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The pull request title clearly and concisely describes the main change: adding content to the 'Where to Contribute' page in the community documentation section.
Linked Issues check ✅ Passed The pull request successfully addresses all coding requirements from issue #4774: adds substantive content explaining different contribution areas (documentation, code, testing, design, community support) and provides guidance on getting started.
Out of Scope Changes check ✅ Passed All changes are scoped to the 'Where to Contribute' onboarding page; the commit message about markdown link formatting is a minor, related documentation improvement with no out-of-scope modifications.
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

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 Dec 25, 2025

Deploy Preview for asyncapi-website ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit 2084834
🔍 Latest deploy log https://app.netlify.com/projects/asyncapi-website/deploys/694e8653ce4e1e000801491f
😎 Deploy Preview https://deploy-preview-4775--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 Dec 25, 2025

Codecov Report

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

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

☔ 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: 1

🧹 Nitpick comments (1)
markdown/docs/community/000-onboarding/where-to-contribute.md (1)

6-77: Content structure and coverage look good.

The document provides a comprehensive onboarding guide that addresses the PR objectives well. Each section clearly explains a contribution area with examples and relevant guidance. The organization is logical, and the tone is welcoming to new contributors.

Consider whether the "Testing and Quality," "Design and UX," and "Community and Support" sections should also list relevant repositories (similar to Documentation and Code sections) to provide more actionable guidance—this may improve discoverability for contributors interested in those areas.

📜 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 a2fae36 and 72fb67e.

📒 Files selected for processing (1)
  • markdown/docs/community/000-onboarding/where-to-contribute.md
🧰 Additional context used
🪛 markdownlint-cli2 (0.18.1)
markdown/docs/community/000-onboarding/where-to-contribute.md

23-23: Bare URL used

(MD034, no-bare-urls)


24-24: Bare URL used

(MD034, no-bare-urls)


39-39: Bare URL used

(MD034, no-bare-urls)


40-40: Bare URL used

(MD034, no-bare-urls)


41-41: Bare URL used

(MD034, no-bare-urls)

⏰ 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). (5)
  • GitHub Check: Redirect rules - asyncapi-website
  • GitHub Check: Header rules - asyncapi-website
  • GitHub Check: Pages changed - asyncapi-website
  • GitHub Check: Test NodeJS PR - windows-latest
  • GitHub Check: Lighthouse CI

@asyncapi-bot
Copy link
Contributor

asyncapi-bot commented Dec 25, 2025

⚡️ Lighthouse report for the changes in this PR:

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

Lighthouse ran on https://deploy-preview-4775--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 (1)
markdown/docs/community/000-onboarding/where-to-contribute.md (1)

74-79: Enhance "How to Get Started" with Slack workspace link.

Line 79 references joining the "AsyncAPI Slack workspace" but doesn't provide a link or joining instructions. Since this section guides first-time contributors to take action, include a direct link or reference to where contributors can access the Slack workspace (e.g., community guidelines page, main README, or invite URL) to make the onboarding flow more seamless.

📜 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 72fb67e and b5ef379.

📒 Files selected for processing (1)
  • markdown/docs/community/000-onboarding/where-to-contribute.md
🧰 Additional context used
🧠 Learnings (3)
📚 Learning: 2025-01-19T04:51:41.255Z
Learnt from: anshgoyalevil
Repo: asyncapi/website PR: 3557
File: tests/fixtures/markdown/check-edit-links-data.js:3-11
Timestamp: 2025-01-19T04:51:41.255Z
Learning: In the AsyncAPI website repository, the test data in `tests/fixtures/markdown/check-edit-links-data.js` intentionally includes inconsistent paths (with and without 'docs' prefix) to verify the script's ability to normalize and handle ambiguous path structures.

Applied to files:

  • markdown/docs/community/000-onboarding/where-to-contribute.md
📚 Learning: 2024-11-13T20:34:18.998Z
Learnt from: bandantonio
Repo: asyncapi/website PR: 3393
File: markdown/blog/2024-Q1-docs-report.md:10-12
Timestamp: 2024-11-13T20:34:18.998Z
Learning: In the markdown files for the AsyncAPI website, avatar image paths should be referenced without the `public` prefix (e.g., use `/img/avatars/...` instead of `/public/img/avatars/...`).

Applied to files:

  • markdown/docs/community/000-onboarding/where-to-contribute.md
📚 Learning: 2025-07-19T20:58:34.040Z
Learnt from: bandantonio
Repo: asyncapi/website PR: 4264
File: markdown/docs/tutorials/getting-started/coming-from-openapi.md:24-25
Timestamp: 2025-07-19T20:58:34.040Z
Learning: In the AsyncAPI website documentation, anchor references to specification sections can use camelCase format (e.g., #serverObject, #parameterObject, #messageObject) even if the actual HTML IDs on the spec page use hyphenated lowercase format. This is acceptable and should not be changed.

Applied to files:

  • markdown/docs/community/000-onboarding/where-to-contribute.md
⏰ 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). (1)
  • GitHub Check: Test NodeJS PR - windows-latest
🔇 Additional comments (1)
markdown/docs/community/000-onboarding/where-to-contribute.md (1)

1-79: Strong onboarding content addition.

The documentation comprehensively addresses all five contribution areas (documentation, code, testing, design, community support) and provides clear entry points for new contributors. The past bare URL issue has been resolved with proper markdown link formatting throughout.

@sammy200-ui
Copy link
Contributor

hello @me-pradeep i would suggest go through contributing guide, AsyncAPI follows a proposal-first approach
After opening an issue and get approval from a maintainer before raising a PR.

@me-pradeep
Copy link
Author

@sammy200-ui Hey bro, I’ve already read the contribution guidelines. As per a message from one of the maintainers on Slack, we can submit a PR without being assigned.
{01028B12-E156-4950-AAB2-85D46E381BF5}

@sammy200-ui
Copy link
Contributor

sammy200-ui commented Dec 26, 2025

@me-pradeep did you read what the maintainer said in that message ? , he said that to that specific issue only, but i do get your point about maintainer not assigning issues here , but you clearly didnt read the guide as you are imposing you did
Screenshot 2025-12-26 at 6 27 15 PM

@me-pradeep
Copy link
Author

@sammy200-ui I’ve already looked at the diagram, but have you seen this comment?
{6F404776-496D-4D98-A517-FD7AEB68F1FB}

@sammy200-ui
Copy link
Contributor

sammy200-ui commented Dec 26, 2025

@me-pradeep i dont know what you are referring in that shot...i think you are confusing assign with approval , you are not to get assigned but you still need approval from the maintainers to begin work, in that shot the maintainer said to start working for the contributor , you are just putting some random screenshots here without getting the context , Not having assignments does not mean PRs can be opened without prior approval. Hoping confusion is clear from my side

@me-pradeep
Copy link
Author

me-pradeep commented Dec 26, 2025

@sammy200-ui Yeah, but how will I know if I’m approved to make a PR? The only way I can see right now is by getting assigned to the issue. So here approval and assigning can bee seen as same.

@sammy200-ui
Copy link
Contributor

@me-pradeep wait for the approval , general time is about a week. maintainer will checkout your issue , then they suggest to go ahead and make a pr or not, hope that helps 😁

@princerajpoot20
Copy link
Member

@me-pradeep Please refrain from raising a PR before the issue is approved by the maintainers.

@me-pradeep
Copy link
Author

@princerajpoot20 okay sir I'll keep that in mind.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[📑 Docs]: Add content to "Where to Contribute" onboarding page

4 participants