Skip to content

Update packages to address GHSA-6rw7-vpxm-498p#44

Merged
AlexJSully merged 1 commit intomainfrom
alexjsully-260102-GHSA-6rw7-vpxm-498p
Jan 2, 2026
Merged

Update packages to address GHSA-6rw7-vpxm-498p#44
AlexJSully merged 1 commit intomainfrom
alexjsully-260102-GHSA-6rw7-vpxm-498p

Conversation

@AlexJSully
Copy link
Member

@AlexJSully AlexJSully requested a review from Copilot January 2, 2026 21:37
@AlexJSully AlexJSully self-assigned this Jan 2, 2026
Copy link

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 PR updates several npm development dependencies to address security vulnerability GHSA-6rw7-vpxm-498p. Additionally, it includes code formatting improvements by removing unnecessary semicolons from inline event handlers throughout the HTML files.

Key changes:

  • Updates development dependencies including @playwright/test, @types/node, eslint, globals, markdownlint-cli, prettier, and workbox-cli
  • Standardizes inline JavaScript event handler formatting by removing trailing semicolons
  • Formats multi-statement event handlers across multiple lines for improved readability

Reviewed changes

Copilot reviewed 3 out of 5 changed files in this pull request and generated no comments.

File Description
package.json Updates development dependency versions to address security vulnerability
index.html Removes trailing semicolons from inline event handlers and formats multi-statement handlers
cgi-bin/Submission_page.html Removes trailing semicolons from inline event handlers and formats multi-statement handlers

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@AlexJSully AlexJSully merged commit dd51d82 into main Jan 2, 2026
5 checks passed
@AlexJSully AlexJSully deleted the alexjsully-260102-GHSA-6rw7-vpxm-498p branch January 2, 2026 21:39
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.

2 participants