fresh icon indicating copy to clipboard operation
fresh copied to clipboard
verified publisher icon denoland

suggest renaming deno tasks

Open liigo opened this issue 2 years ago • 2 comments

  • deno task start may be rename to deno task dev
  • deno task preview may be rename to deno task serve

reason: cleaner description.

ref: npm run dev npm run serve.

liigo avatar Dec 27 '23 10:12 liigo

ref: npm run dev npm run serve.

Can you elaborate on where you got those references from? I feel like this issue is a bit missing the reasons on "why" those names are preferable. I'm certainly open to changing the names of the tasks.

marvinhagemeister avatar Dec 27 '23 10:12 marvinhagemeister

@marvinhagemeister

They don't clearly express which scenario to be used for. deno task start is for dev/debug scenario, but 'start' may be misleading (to start a production version). deno task preview is for production/release scene, but 'preview' is obvious misleading (to preview a develop version). 'dev' / 'serve' is cleaner in semantic than 'start' / 'preview', IMO.

liigo avatar Dec 28 '23 06:12 liigo

We renamed the tasks for the upcoming Fresh 2 release to

  • dev: Run development server
  • start: Run production server
  • build: Build production app, same as before

marvinhagemeister avatar Jun 05 '24 08:06 marvinhagemeister