Skip to content

update path for runner#1463

Merged
gugu merged 1 commit into
mainfrom
run-script-path-fix
Dec 9, 2025
Merged

update path for runner#1463
gugu merged 1 commit into
mainfrom
run-script-path-fix

Conversation

@gugu

@gugu gugu commented Dec 9, 2025

Copy link
Copy Markdown
Contributor

No description provided.

Copilot AI review requested due to automatic review settings December 9, 2025 10:42

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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.

Comment thread backend/runner.sh
@@ -1,5 +1,5 @@
#!/bin/bash
node --enable-source-maps dist/main.js &
node --enable-source-maps dist/src/main.js &

Copilot AI Dec 9, 2025

Copy link

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