-
Notifications
You must be signed in to change notification settings - Fork 138
FIX:Updated Operators and datatypes of SQL #1045
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
The previous version of the introduction article had an illogical flow for new learners, jumping between concepts without building a proper foundation. SEO elements were also minimal. This commit completely restructures the content to be more beginner-friendly and impactful: - Introduces the concept of Version Control first to establish the "why". - Clearly distinguishes between Git (the tool) and GitHub (the platform). - Improves SEO with a keyword-rich title, meta description, and tags. - Rewrites core concepts like 'Repository' and 'Pull Request' for better clarity. - Organizes the content with a logical heading structure.
The previous version of the introduction article had an illogical flow for new learners, jumping between concepts without building a proper foundation. SEO elements were also minimal. This commit completely restructures the content to be more beginner-friendly and impactful: - Introduces the concept of Version Control first to establish the "why". - Clearly distinguishes between Git (the tool) and GitHub (the platform). - Improves SEO with a keyword-rich title, meta description, and tags. - Rewrites core concepts like 'Repository' and 'Pull Request' for better clarity. - Organizes the content with a logical heading structure.
1.The entire meta block was commented out and has been fixed and enabled. 2.The og:url, og:image, and og:description tags were pointing to an incorrect article about the MERN stack. They now correctly point to the DeepMind article's assets and have an accurate, well-written description. 3.The primary description field was rewritten to be grammatically correct and more compelling.
Updated content to improve clarity and consistency throughout the article, including sections on energy efficiency, healthcare applications, and AI technology.
Fix: Proofread and Correct Metadata in DeepMind Article
1.Frontmatter and Naming Standardization: Consistent Naming: Standardized the program name to "Google Student Ambassador" throughout the article, removing confusion from variations like "Campus Ambassador" and "Gemini Ambassador." Synced Metadata: Updated the title, slug, and id in the frontmatter to be consistent with the main <h1> heading and article content. The new slug is /google-student-ambassador-program-2025. 2. Content and Factual Accuracy: Corrected Time Commitment: Resolved the contradiction in the FAQ. Both Q2 and Q3 now correctly state the time commitment as "5–7 hours per week." Removed Empty Section: Deleted the "Got Questions & What Happens Next?" section, as it contained no actionable information and detracted from the article's professional tone. 3.Accessibility and Code Quality: Added Alt Text: Provided descriptive alt text for both images, ensuring the article is accessible to screen readers. Removed Unused Import: Deleted an unused Layout component import to clean up the code.
1.Frontmatter and Naming Standardization: Consistent Naming: Standardized the program name to "Google Student Ambassador" throughout the article, removing confusion from variations like "Campus Ambassador" and "Gemini Ambassador." Synced Metadata: Updated the title, slug, and id in the frontmatter to be consistent with the main <h1> heading and article content. The new slug is /google-student-ambassador-program-2025. 2. Content and Factual Accuracy: Corrected Time Commitment: Resolved the contradiction in the FAQ. Both Q2 and Q3 now correctly state the time commitment as "5–7 hours per week." Removed Empty Section: Deleted the "Got Questions & What Happens Next?" section, as it contained no actionable information and detracted from the article's professional tone. 3. Accessibility and Code Quality: Added Alt Text: Provided descriptive alt text for both images, ensuring the article is accessible to screen readers. Removed Unused Import: Deleted an unused Layout component import to clean up the code.
1.Activated Social Media Metadata: The entire meta block containing Open Graph and Twitter card information was commented out. This has been uncommented, enabling rich previews when the article is shared on social platforms. 2.Added Canonical URL: Populated the empty canonical_url field to prevent duplicate content issues and improve search engine indexing. 3.Aligned Page Titles: The frontmatter title has been updated to match the on-page <h1> heading for better SEO alignment and user experience. The sidebar_label was also shortened for better UI fit. 4.Improved Image Accessibility: Enhanced the alt text for the architecture diagram to be more descriptive for users with screen readers.
1.The entire meta block containing Open Graph and Twitter card information was commented out. This has been uncommented, enabling rich previews when the article is shared on social platforms. 2.Populated the empty canonical_url field to prevent duplicate content issues and improve search engine indexing. 3.The frontmatter title has been updated to match the on-page <h1> heading for better SEO alignment and user experience. The sidebar_label was also shortened for better UI fit. 4.Enhanced the alt text for the architecture diagram to be more descriptive for users with screen readers.
This update refactors the sql-operators tutorial to improve technical accuracy, readability, and structural correctness. Key changes include: 1.Corrected "Operator Precedence" Example: The original examples in this section were flawed as they produced the same output, failing to demonstrate the risk of not using parentheses. This section has been rewritten with new queries that produce different results to clearly illustrate the concept. 2.Fixed LIKE Logic: Corrected the LIKE pattern matching example for "Alice" from ___ice% (3 underscores) to __ice% (2 underscores). 3.Added Missing Outputs: Provided the missing query outputs for the BETWEEN, IN, and corrected LIKE examples so users can see the result of every query. 4.Added Syntax Highlighting: Applied sql language highlighting to all code blocks for better readability. Fixed Semantic Headings: Changed main section headings from H4 (e.g., 📘) to H2 (##) for proper document structure. Standardized SQL Style: Converted all SQL data types (e.g., INT, VARCHAR) to UPPERCASE for consistency. Improved Clarity: Added a note that BETWEEN is inclusive and merged the redundant "Tips" and "Common Mistakes" sections.
1.Refactored All Examples: Replaced the multiple, fragmented CREATE TABLE statements with a single, unified students table example. This makes the tutorial more practical and easier to follow, as all common data types are shown working together. Standardized Code Blocks: Replaced all broken :::info blocks with proper sql syntax-highlighted code blocks and clean, separate Markdown tables for query outputs. Expanded Date/Time Section: Added the essential DATETIME and TIMESTAMP types, which were missing, and explained their common use cases (like the last_login example). Improved Numeric Guidance: Changed the gpa example from FLOAT to DECIMAL(3, 2) to teach the best practice for handling precise values (like grades or money) and strengthened the warning against using FLOAT. Integrated UNSIGNED Concept: Removed the separate, confusing UNSIGNED example and integrated the concept directly into the main students table by making the student_id an INT UNSIGNED. Cleaned Up Formatting: Consolidated the redundant tags and keywords in the frontmatter, added horizontal rules (---) between sections for readability, and standardized all SQL data types (e.g., INT, VARCHAR) to UPPERCASE for consistency.
|
@DIYA-bot 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! 😊 |
|
|
||
|
|
||
|
|
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.
Front matter should begin immediately without blank lines - lines 1-3 contain unnecessary blank lines before the --- separator.
View Details
📝 Patch Details
diff --git a/docs/sql/SQL-basics/datatypes.md b/docs/sql/SQL-basics/datatypes.md
index d371913..c44a2f1 100644
--- a/docs/sql/SQL-basics/datatypes.md
+++ b/docs/sql/SQL-basics/datatypes.md
@@ -1,6 +1,3 @@
-
-
-
---
id: sql-datatypes
title: SQL Data Types
@@ -35,11 +32,11 @@ Data types are super important because they:
Imagine a `students` table. Data types ensure the `student_id` is a number, `first_name` is text, and `enrollment_date` is a date.
-Here’s a handy overview to get you started:
+Here's a handy overview to get you started:
:::tip Pro Tip
-Always pick the smallest, most appropriate data type for your data. For example, don’t use a text type for numbers you want to do math on!
+Always pick the smallest, most appropriate data type for your data. For example, don't use a text type for numbers you want to do math on!
:::
---
@@ -112,7 +109,7 @@ In our students table, we used INT UNSIGNED for student_id. This is perfect beca
Use UNSIGNED for any number column that can never be negative (like an ID, age, or stock quantity).
:::
✅ What You've Learned
-Great job! You’ve explored the most important SQL data types:
+Great job! You've explored the most important SQL data types:
Numeric Types: INT, DECIMAL (for precision), and FLOAT.
String Types: CHAR (fixed) vs. VARCHAR (variable).
Date/Time Types: DATE, DATETIME, and TIMESTAMP.
Analysis
YAML front matter parsing fails due to blank lines before --- separator
What fails: Docusaurus YAML front matter parser doesn't recognize front matter when preceded by blank lines in docs/sql/SQL-basics/datatypes.md
How to reproduce:
# Test front matter parsing with regex used by Docusaurus
node -e "
const fs = require('fs');
const content = fs.readFileSync('docs/sql/SQL-basics/datatypes.md', 'utf8');
const regex = /^---\n([\s\S]*?)\n---/;
const match = content.match(regex);
console.log('Front matter detected:', !!match);
"Result: Returns Front matter detected: false when file starts with blank lines before ---
Expected: Should return Front matter detected: true for proper YAML parsing per Docusaurus markdown documentation and YAML front matter standards which require front matter to begin on the first line
| GitHub,git,version control | ||
| Introduction of GitHub, | ||
| What is GitHub, | ||
| Why learn GitHub, | ||
| What is GitHub, open source | ||
| Why learn GitHub,beginner guide | ||
| How to use GitHub, | ||
| ] | ||
| description: In this tutorial, you will learn about GitHub, its importance, what is GitHub from Scratch, how to use GitHub, steps to start using GitHub, and more. | ||
| --- | ||
| description: New to GitHub? This beginner's guide explains what Git and GitHub are, why they're essential for developers, and how to start your open-source journey. Learn key concepts like repositories, commits, and pull requests. |
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.
| GitHub,git,version control | |
| Introduction of GitHub, | |
| What is GitHub, | |
| Why learn GitHub, | |
| What is GitHub, open source | |
| Why learn GitHub,beginner guide | |
| How to use GitHub, | |
| ] | |
| description: In this tutorial, you will learn about GitHub, its importance, what is GitHub from Scratch, how to use GitHub, steps to start using GitHub, and more. | |
| --- | |
| description: New to GitHub? This beginner's guide explains what Git and GitHub are, why they're essential for developers, and how to start your open-source journey. Learn key concepts like repositories, commits, and pull requests. | |
| GitHub,git,version control, | |
| Introduction of GitHub, | |
| What is GitHub, open source, | |
| Why learn GitHub,beginner guide, | |
| How to use GitHub, | |
| ] | |
| description: New to GitHub? This beginner's guide explains what Git and GitHub are, why they're essential for developers, and how to start your open-source journey. Learn key concepts like repositories, commits, and pull requests. | |
| --- |
Missing commas in YAML array causes invalid syntax, and front matter is missing its closing --- separator.
View Details
Analysis
YAML front matter parsing errors in intro-github.md
What fails: YAML front matter in docs/Technical/intro-github.md has invalid flow sequence syntax and missing closing separator, causing incorrect tag parsing and front matter extraction failure
How to reproduce:
# Test YAML parsing with js-yaml
node -e "const yaml=require('js-yaml'); console.log(yaml.load(frontMatter).tags)"
# Results in concatenated tags: ["GitHub", "git", "version control Introduction of GitHub"]
# Test front matter extraction
node -e "console.log('---\\ndata\\n---\\ncontent'.match(/^---\\s*\\n([\\s\\S]*?)\\n---/))"
# Returns null due to missing closing ---Result: Tags array incorrectly parses as 6 items instead of 9, with values concatenated (e.g., "version control Introduction of GitHub"). Front matter extraction fails completely without closing --- separator.
Expected: Tags should parse as 9 separate items. Front matter should be properly delimited per YAML specification requiring commas in flow sequences and Jekyll front matter requiring closing ---.
This commit refines the "Spark Architecture Explained" blog post, addressing a major technical inaccuracy and cleaning up frontmatter metadata. Key changes: Fix (Memory Model): Rewrites the "Memory Management" section to accurately describe the modern Unified Memory Model used since Spark 1.6. This removes the outdated and incorrect description of the old static 60/20/20 memory split. 1.Fix (Frontmatter): Deletes the entire commented-out meta: block, which contained incorrect placeholder content about DeepMind and MERN stack. 2.Chore (Frontmatter): Removes the empty canonical_url key. 3.Style (Code): Adds the scala language specifier to code blocks to enable proper syntax highlighting. 4.Style (Accessibility): Improves the alt text for the architecture diagram for better accessibility.
docs(spark): Correct Spark memory model and fix post metadata
| @@ -1,4 +1,3 @@ | |||
| --- | |||
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.
The frontmatter is malformed - the opening --- delimiter is missing. The file starts with title: on line 1 instead of ---.
View Details
📝 Patch Details
diff --git a/blog/spark-architecture/index.md b/blog/spark-architecture/index.md
index e4b0784..5d698a1 100644
--- a/blog/spark-architecture/index.md
+++ b/blog/spark-architecture/index.md
@@ -1,3 +1,4 @@
+---
title: "Spark Architecture Explained"
authors: [Aditya-Singh-Rathore,sanjay-kv]
sidebar_label: "Spark Architecture Explained"
Analysis
Missing frontmatter opening delimiter in blog/spark-architecture/index.md breaks metadata parsing
What fails: File blog/spark-architecture/index.md frontmatter missing opening --- delimiter prevents Docusaurus from parsing blog metadata (title, authors, tags, date, description)
How to reproduce:
# Test frontmatter parsing with gray-matter (Docusaurus's parser):
node -e "
const matter = require('gray-matter');
const fs = require('fs');
const content = fs.readFileSync('blog/spark-architecture/index.md', 'utf8');
console.log('Metadata parsed:', Object.keys(matter(content).data).length > 0);
"Result: Returns false - no metadata fields recognized. File starts with title: instead of proper ---\ntitle: format
Expected: Should return true with 7 metadata fields (title, authors, tags, etc.) parsed per Docusaurus frontmatter specification
This commit applies several copy-edits to the "7 Steps to Streamline Your UX Design Process" article to improve readability, flow, and remove redundant content. Section 1 (Research): De-duplicated repetitive content and merged it into a single, stronger introductory section. Section 4 (Refine): Deleted a misplaced sentence about prototypes that belonged in the next step. Section 6 (Feedback): Removed a weak introductory paragraph to make the section more direct. Section 7 (Launch): Streamlined the content by removing generic opening paragraphs and a redundant concluding paragraph that was already covered in the "Takeaway" section. Footer: Fixed a typo (a stray "f") in the contact email address. Frontmatter: Removed a non-standard space before the truncate comment.
Here is a commit message and description for the changes you made. Subject: docs(ux): Refine and streamline UX design process article Description: This commit applies several copy-edits to the "7 Steps to Streamline Your UX Design Process" article to improve readability, flow, and remove redundant content. Section 1 (Research): De-duplicated repetitive content and merged it into a single, stronger introductory section. Section 4 (Refine): Deleted a misplaced sentence about prototypes that belonged in the next step. Section 6 (Feedback): Removed a weak introductory paragraph to make the section more direct. Section 7 (Launch): Streamlined the content by removing generic opening paragraphs and a redundant concluding paragraph that was already covered in the "Takeaway" section. Footer: Fixed a typo (a stray "f") in the contact email address. Frontmatter: Removed a non-standard space before the truncate comment.
1.Fix (Frontmatter): Corrected a spelling error in the tags (from "Desgin" to "Design"). 2.Fix (Frontmatter): Replaced the placeholder "Google DeepMind" description with a summary that is relevant to the article's content. 3.Chore (Frontmatter): Deleted the entire commented-out meta: block, as it was placeholder content from a template. 4.Chore (Frontmatter): Removed the empty canonical_url key. 5.Refactor (Content): Removed multiple repetitive paragraphs and sections. The same core points about AI's role, the challenges for designers, and the importance of human-centricity were repeated in the intro, a dedicated section, and the conclusion. These have been consolidated for a more concise and impactful article. 6.Chore (Formatting): Removed a stray em-space before the `` comment and an empty bullet point.
1.Fix (Frontmatter): Corrected a spelling error in the tags (from "Desgin" to "Design"). 2.Fix (Frontmatter): Replaced the placeholder "Google DeepMind" description with a summary that is relevant to the article's content. 3.Chore (Frontmatter): Deleted the entire commented-out meta: block, as it was placeholder content from a template. 4.Chore (Frontmatter): Removed the empty canonical_url key. 5.Refactor (Content): Removed multiple repetitive paragraphs and sections. The same core points about AI's role, the challenges for designers, and the importance of human-centricity were repeated in the intro, a dedicated section, and the conclusion. These have been consolidated for a more concise and impactful article. 6.Chore (Formatting): Removed a stray em-space before the `` comment and an empty bullet point.
This commit provides a major cleanup and refactoring of the "10 Steps to Land a Job in UI/UX Design" article. The changes address metadata errors, structural issues, and significant content redundancy. 1.Metadata Fixes: Corrected a spelling error in the tags (Desgin -> Design), replaced an irrelevant AI tag, and removed placeholder/empty keys (meta, canonical_url). 2.Structural Improvements: Moved the main article introduction to the top (it was incorrectly placed within the first step) and added numbering to the 10 step headings for better clarity. 3.Content Refactoring: Removed repetitive concluding paragraphs from nearly every step. This makes the article much more concise and easier to read. 4.Content Correction: Fixed incorrect step numbers mentioned within the text and replaced the "Final Verdict" section's content, which was a copy-paste error from a different article.
This commit provides a major cleanup and refactoring of the "10 Steps to Land a Job in UI/UX Design" article. The changes address metadata errors, structural issues, and significant content redundancy. Metadata Fixes: Corrected a spelling error in the tags (Desgin -> Design), replaced an irrelevant AI tag, and removed placeholder/empty keys (meta, canonical_url). Structural Improvements: Moved the main article introduction to the top (it was incorrectly placed within the first step) and added numbering to the 10 step headings for better clarity. Content Refactoring: Removed repetitive concluding paragraphs from nearly every step. This makes the article much more concise and easier to read. Content Correction: Fixed incorrect step numbers mentioned within the text and replaced the "Final Verdict" section's content, which was a copy-paste error from a different article.
|
|
||
|
|
||
| - --- |
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.
The file has invalid frontmatter syntax with a malformed - --- marker on line 3 that breaks the YAML frontmatter block.
View Details
📝 Patch Details
diff --git a/docs/sql/SQL-basics/Operators.md b/docs/sql/SQL-basics/Operators.md
index 50fd12f..a886f9b 100644
--- a/docs/sql/SQL-basics/Operators.md
+++ b/docs/sql/SQL-basics/Operators.md
@@ -1,6 +1,4 @@
-
-
-- ---
+---
id: sql-operators
title: SQL Operators
sidebar_label: SQL Operators
@@ -66,4 +64,4 @@ INSERT INTO products VALUES (1, 15.99, 10), (2, 25.50, 5);
SELECT id, price, quantity,
price * quantity AS total_value,
price + 2.00 AS price_with_tax
-FROM products;
+FROM products;
\ No newline at end of file
Analysis
Invalid YAML frontmatter syntax breaks Docusaurus parsing
What fails: docs/sql/SQL-basics/Operators.md has malformed frontmatter with - --- delimiter instead of ---, causing markdown parser to fail
How to reproduce:
npm run build
# Would cause frontmatter parsing error for Operators.mdResult: The frontmatter block starting with - --- is invalid YAML syntax that prevents proper metadata extraction
Expected: Frontmatter must start with --- on the first line per Docusaurus documentation - "Front matter is provided at the very top of the file, enclosed by three dashes ---"
| (3, 'Charlie', 'Brown', 2.50, '2023-08-14', FALSE, NULL); | ||
|
|
||
| SELECT * FROM students; | ||
| Now, let's break down the types we used. |
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.
| Now, let's break down the types we used. | |
| ``` | |
| Now, let's break down the types we used. | |
Missing blank line between code block and text causes formatting to run together, and line 70 has run-on text that should be separated.
View Details
Analysis
Unclosed SQL code block breaks markdown formatting in datatypes.md
What fails: SQL code block starting at line 54 with ```sql never gets closed, causing all subsequent content to render as code instead of formatted markdown
How to reproduce:
# View the file in any markdown renderer
cat docs/sql/SQL-basics/datatypes.md | grep -A 20 "SELECT \* FROM students;"Result: Text "Now, let's break down the types we used." and heading "## Numeric Data Types (Numbers)" render as plain code text instead of formatted markdown
Expected: Code block should be properly closed with ``` followed by blank lines, allowing subsequent text and headings to render correctly per CommonMark spec requirements for fenced code blocks
sheikhlimon
left a comment
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.
Thanks for your PR! For future contributions, it’s best to create a separate branch instead of working directly on main. This helps keep things organized and makes reviews easier.
|
@DIYA-bot check the contributing guidelines |
|
@DIYA-bot closing this as i havnt assigned any related issues and the docs change is across 10 blogs instead of sql |
|
Ok> @DIYA-bot check the contributing guidelines |
Description
Fixes # (issue)
Type of Change
Changes Made
Dependencies
Checklist
npm run buildand attached screenshot(s) in this PR.