whoiskatrin / sql-translator

SQL Translator is a tool for converting natural language queries into SQL code using artificial intelligence. This project is 100% free and open source.
https://www.sqltranslate.app/
MIT License
4.16k stars 356 forks source link

correct command for starting dev server #24

Closed cookabc closed 1 year ago

cookabc commented 1 year ago

I think starting development server should be: npm run dev

If you want to use npm run start, you should run npm run build first

vercel[bot] commented 1 year ago

Someone is attempting to deploy a commit to a Personal Account owned by @whoiskatrin on Vercel.

@whoiskatrin first needs to authorize it.

whoiskatrin commented 1 year ago

Please refer to this page, https://nextjs.org/docs/getting-started

Npm start is a default setup for a project

cookabc commented 1 year ago

Hi @whoiskatrin , thanks for the link. But the command npm start just does not work:

image

Is there anything I missed?

shuabrannigan commented 1 year ago

I think starting development server should be: npm run dev

If you want to use npm run start, you should run npm run build first

this is true, readme should reflect this npm start only works when application has been built for production

cookabc commented 1 year ago

you have to run 'npm run build' to build production 'npm start' then works as intended

maybe an update to the readme to make that clearer?

Thanks @shuabrannigan , you are right.

Since it's to start a development server, I think npm run dev is the proper command. Anyway, I added npm run build into README before npm start.