Skip to content

Conversation

@gugu
Copy link
Contributor

@gugu gugu commented Dec 9, 2025

No description provided.

Copilot AI review requested due to automatic review settings December 9, 2025 10:42
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 the path to the compiled main.js file in the runner script to reflect the correct build output structure of the NestJS application.

  • Updated the node executable path from dist/main.js to dist/src/main.js to align with the TypeScript build output structure

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

@@ -1,5 +1,5 @@
#!/bin/bash
node --enable-source-maps dist/main.js &
node --enable-source-maps dist/src/main.js &
Copy link

Copilot AI Dec 9, 2025

Choose a reason for hiding this comment

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

The path is being updated to dist/src/main.js in this file, but backend/package.json line 23 still references node dist/main in the start:prod script. These paths should be consistent. Consider updating the start:prod script to "start:prod": "node dist/src/main" for consistency.

Copilot uses AI. Check for mistakes.
@gugu gugu merged commit 74980c3 into main Dec 9, 2025
24 checks passed
@gugu gugu deleted the run-script-path-fix branch December 9, 2025 11:02
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