Skip to content

Change script tag to a level 2 heading#213

Open
gurpaul-kochhar wants to merge 1 commit intomainfrom
gurpaul-kochhar-patch-2
Open

Change script tag to a level 2 heading#213
gurpaul-kochhar wants to merge 1 commit intomainfrom
gurpaul-kochhar-patch-2

Conversation

@gurpaul-kochhar
Copy link
Collaborator

No description provided.

Copilot AI review requested due to automatic review settings January 16, 2026 01:34
Copy link
Contributor

Copilot AI left a 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 pull request adds markdown level 2 heading syntax (## ) before a script tag that loads JavaScript for fetching student count data. This change breaks the functionality of the script by treating it as a heading instead of executable HTML.

Changes:

  • Modified the script tag on line 82 to be prefixed with markdown heading syntax

<script async defer src="https://buttons.github.io/buttons.js"></script>

<script src="_static/fetch_student_count.js"></script>
## <script src="_static/fetch_student_count.js"></script>
Copy link

Copilot AI Jan 16, 2026

Choose a reason for hiding this comment

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

The script tag should not be prefixed with markdown heading syntax. This is an HTML script element that needs to execute JavaScript to fetch and display student counts dynamically. Adding "## " before it will prevent the script from functioning correctly and may render it as visible text on the page instead of executing it. The script tag should remain as plain HTML without any markdown formatting.

Suggested change
## <script src="_static/fetch_student_count.js"></script>
<script src="_static/fetch_student_count.js"></script>

Copilot uses AI. Check for mistakes.
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