Skip to content

Add missing semicolons for ASI consistency#59

Merged
renatoathaydes merged 3 commits intoIS-10950-previewer-listingfrom
copilot/sub-pr-57
Mar 17, 2026
Merged

Add missing semicolons for ASI consistency#59
renatoathaydes merged 3 commits intoIS-10950-previewer-listingfrom
copilot/sub-pr-57

Conversation

Copy link
Copy Markdown

Copilot AI commented Jan 28, 2026

Addresses code review feedback to avoid automatic semicolon insertion in src/identity-server/previewer.js.

Changes

  • Added semicolons to 5 statements missing them (lines 17, 202, 207, 237, 243)
  • Ensures consistency with existing code style where 92%+ of statements use explicit semicolons
// Before
procArgs.push(process.env.LISTING_TEMPLATE)

// After
procArgs.push(process.env.LISTING_TEMPLATE);

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Copilot AI and others added 2 commits January 28, 2026 07:10
Co-authored-by: markoweb <2862929+markoweb@users.noreply.github.com>
Co-authored-by: markoweb <2862929+markoweb@users.noreply.github.com>
Copilot AI changed the title [WIP] Update reading listing template based on feedback from IS-10950 Add missing semicolons for ASI consistency Jan 28, 2026
Copilot AI requested a review from markoweb January 28, 2026 07:14
@renatoathaydes renatoathaydes marked this pull request as ready for review March 17, 2026 08:00
@renatoathaydes renatoathaydes merged commit 396902f into IS-10950-previewer-listing Mar 17, 2026
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.

4 participants