Skip to content
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

Update School_Queries.sql #1

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Imran-imtiaz48
Copy link

The changes I made to the script:

  1. Formatted SQL Keywords: Changed SQL keywords to uppercase for better readability.
  2. Used INSERT INTO ... VALUES: Grouped multiple INSERT statements into a single INSERT INTO ... VALUES statement for each table to make the code more concise.
  3. Improved Comments: Made comments more descriptive and consistent.
  4. Consistency in Formatting: Ensured consistent spacing and formatting throughout the script.
  5. Date Format Consistency: Ensured dates are in the same format YYYY-MM-DD.
  6. Corrected Typo: Fixed the spelling of 'Python' in the CourseMaster table.
  7. Removed Redundant Commands: Removed redundant GO commands after data insertion and kept the essential ones for logical separation.

These changes improve readability, maintainability, and ensure consistency throughout the script.

The changes I made to the script:

1. **Formatted SQL Keywords**: Changed SQL keywords to uppercase for better readability.
2. **Used `INSERT INTO ... VALUES`**: Grouped multiple `INSERT` statements into a single `INSERT INTO ... VALUES` statement for each table to make the code more concise.
3. **Improved Comments**: Made comments more descriptive and consistent.
4. **Consistency in Formatting**: Ensured consistent spacing and formatting throughout the script.
5. **Date Format Consistency**: Ensured dates are in the same format `YYYY-MM-DD`.
6. **Corrected Typo**: Fixed the spelling of 'Python' in the CourseMaster table.
7. **Removed Redundant Commands**: Removed redundant `GO` commands after data insertion and kept the essential ones for logical separation.

These changes improve readability, maintainability, and ensure consistency throughout the script.
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.

1 participant