Skip to content

Conversation

@Ansita20
Copy link

Notes for Reviewers

This PR fixes #835

CONTRIBUTING.md
Errors Fixed:

❌ Missing prerequisites context and version requirements
❌ No step-by-step setup instructions
❌ Incomplete contribution workflow and PR guidelines
❌ Missing style guidelines and commit message conventions

Improvements: Added table of contents, detailed setup guide, git workflow with commands, PR best practices, and community support section.

README.md
Errors Fixed:

❌ No table of contents or navigation
❌ Missing visual preview of documentation site
❌ Incomplete contributing guidelines (no image/alert examples)
❌ Minimal community engagement information

Improvements: Added TOC, documentation GIF preview, target audience matrix, detailed setup instructions, collapsible documentation structure sections, image/alert syntax examples with tables, expanded community resources, and professional footer.

Total Impact: Both files transformed from basic docs to comprehensive, professional guides with better navigation, practical examples, and complete information for all user types.

@netlify
Copy link

netlify bot commented Oct 15, 2025

Deploy Preview for bejewelled-pegasus-b0ce81 ready!

Name Link
🔨 Latest commit 2d2a243
🔍 Latest deploy log https://app.netlify.com/projects/bejewelled-pegasus-b0ce81/deploys/68f215dd91cbd5000857338c
😎 Deploy Preview https://deploy-preview-839--bejewelled-pegasus-b0ce81.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.

CONTRIBUTING.md Outdated

## <a name="contributing-docs">Documentation Contribution Flow</a>
Please contribute! Layer5 documentation uses Jekyll and GitHub Pages to host docs sites. Learn more about [Layer5's documentation framework](https://docs.google.com/document/d/17guuaxb0xsfutBCzyj2CT6OZiFnMu9w4PzoILXhRXSo/edit?usp=sharing). The process of contributing follows this flow:
Improve the contributing documentation to be more professional,
Copy link
Member

@leecalcote leecalcote Oct 16, 2025

Choose a reason for hiding this comment

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

These aren't the intended changes, but notes from your AI session. Please retry, @Ansita20.

Copy link
Member

Choose a reason for hiding this comment

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

Make sure to sign-off on your commits, too. See the doc that you are conceptually improving for instructions, @Ansita20

Copy link
Author

Choose a reason for hiding this comment

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

yes sir, I will retry.

Copy link
Author

Choose a reason for hiding this comment

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

Sir, Sorry for the previous carelessness. I have fixed the mistake I made. Can you please check the changes.

Signed-off-by: Ansita-1 <ansita6050@gmail.com>
@leecalcote leecalcote requested a review from Vincamine October 24, 2025 19:44
@leecalcote
Copy link
Member

@Vincamine review, please.

@Rajesh-Nagarajan-11
Copy link
Member

Thank you for your contribution!
Let's discuss this during the website call today at 5:30 PM IST | 7 AM CT
Add it as an agenda item to the meeting minutes, if you would 😄

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.

[Doc] Improve Documentation for Academy

3 participants