-
Notifications
You must be signed in to change notification settings - Fork 111
Added docs for all types of SQL joins #461
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
@Adez017 is attempting to deploy a commit to the recode Team on Vercel. A member of the Team first needs to authorize it. |
Thank you for submitting your pull request! 🙌 We'll review it as soon as possible. The estimated time for response is 5–8 hrs. In the meantime, please provide all necessary screenshots and make sure you run - npm build run , command and provide a screenshot, a video recording, or an image of the update you made below, which helps speed up the review and assignment. If you have questions, reach out to LinkedIn. Your contributions are highly appreciated!😊 Note: I maintain the repo issue every day twice at 8:00 AM IST and 9:00 PM IST. If your PR goes stale for more than one day, you can tag and comment on this same issue by tagging @sanjay-kv. We are here to help you on this journey of open source. Consistent 20 contributions are eligible for sponsorship 💰 🎁 check our list of amazing people we sponsored so far: GitHub Sponsorship. ✨ 📚Your perks for contribution to this community 👇🏻
If there are any specific instructions or feedback regarding your PR, we'll provide them here. Thanks again for your contribution! 😊 |
Hi @sanjay-kv , @iitzIrFan please take a look and i also request you to add level3 as if possible |
@Adez017 Do the above changes suggested by co-pilot rest is fine. |
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Hi @iitzIrFan , i had made the changes take a look |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR adds comprehensive documentation for SQL joins to the site. It includes a complete section covering all major join types with detailed examples, syntax explanations, and practical use cases for each join operation.
Key changes:
- Added a new "SQL Joins" sidebar section with 7 new documentation pages
- Provides detailed coverage of INNER JOIN, LEFT JOIN, RIGHT JOIN, FULL OUTER JOIN, CROSS JOIN, and SELF JOIN
- Includes an introductory page explaining SQL joins concepts and when to use each type
Reviewed Changes
Copilot reviewed 8 out of 15 changed files in this pull request and generated 3 comments.
Show a summary per file
File | Description |
---|---|
sidebars.ts | Added new "SQL Joins" category with navigation structure for all join documentation pages |
docs/sql/SQL-joins/intro-sql-joins.md | Introduction page explaining SQL joins concepts and overview of all join types |
docs/sql/SQL-joins/inner-join.md | Comprehensive documentation for INNER JOIN with examples and best practices |
docs/sql/SQL-joins/left-join.md | Detailed LEFT JOIN documentation including practical examples and NULL handling |
docs/sql/SQL-joins/right-join.md | RIGHT JOIN documentation with syntax, examples, and comparison to LEFT JOIN |
docs/sql/SQL-joins/full-outer-join.md | FULL OUTER JOIN documentation with database compatibility notes and alternatives |
docs/sql/SQL-joins/cross-join.md | CROSS JOIN documentation with performance considerations and practical examples |
docs/sql/SQL-joins/self-join.md | SELF JOIN documentation covering hierarchical data and comparison scenarios |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
|
||
], | ||
}; | ||
|
Copilot
AI
Aug 29, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Removed trailing whitespace, which is good practice for clean code formatting.
Copilot uses AI. Check for mistakes.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM ! @Adez017
@sanjay-kv Ready for Merge.
Any Updates ?? @sanjay-kv |
is there Any updates regarding this PR @iitzIrFan @sanjay-kv |
@sanjay-kv , also look into this PR |
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Hi @sanjay-kv , please add the gssoc related lables to this PR and also some of my merged PR |
please add the gssoc related label to this PR @sanjay-kv |
Description
Added docs for all sql joins with examples and summaries and explaining each one in detail
Fixes #372
Type of Change
Changes Made
Added new section in Sql docs as sql joins
added files for each type of join
Dependencies
Checklist
npm run build
and attached scrrenshot in this PR.