Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

ng server runs ng serve #10334

Closed
Toxicable opened this issue Apr 14, 2018 · 2 comments · Fixed by #10401
Closed

ng server runs ng serve #10334

Toxicable opened this issue Apr 14, 2018 · 2 comments · Fixed by #10401
Labels
P3 An issue that is relevant to core functions, but does not impede progress. Important, but not urgent
Milestone
v6.0.0

Comments

@Toxicable
Copy link

Versions

Angular CLI: 6.0.0-rc.4
Node: 8.11.1
OS: linux x64
Angular: 6.0.0-rc.5
... animations, common, compiler, compiler-cli, core, forms
... http, language-service, platform-browser
... platform-browser-dynamic, router

Package                           Version
-----------------------------------------------------------
@angular-devkit/architect         0.5.6
@angular-devkit/build-angular     0.5.6
@angular-devkit/build-optimizer   0.5.6
@angular-devkit/core              0.5.6
@angular-devkit/schematics        0.5.6
@angular/cli                      6.0.0-rc.4
@ngtools/json-schema              1.1.0
@ngtools/webpack                  6.0.0-rc.4
@schematics/angular               0.5.6
@schematics/update                0.5.6
rxjs                              6.0.0-uncanny-rc.7
typescript                        2.7.2
webpack                           4.5.0

Repro steps

  • ng new my proj
  • cd prod
  • yarn ng server

Observed behavior

ng serve is run

Desired behavior

Nothing should be run.

It's confusing since there's another target called server
@hansl hansl added this to the v6.0.0 milestone Apr 15, 2018
@hansl hansl added the P5 The team acknowledges the request but does not plan to address it, it remains open for discussion label Apr 15, 2018
@alan-agius4 alan-agius4 mentioned this issue Apr 20, 2018
Merged
@alan-agius4
Copy link
Collaborator

I found this quite confusing as well

@neta-kedem neta-kedem mentioned this issue Apr 23, 2018
Closed
@hansl hansl added P3 An issue that is relevant to core functions, but does not impede progress. Important, but not urgent and removed P5 The team acknowledges the request but does not plan to address it, it remains open for discussion labels Apr 23, 2018
hansl pushed a commit that referenced this issue Apr 23, 2018
@alan-agius4 @hansl
feat(@angular/cli): remove server alias for serve
bc56efa 
The alias has been removed as it's quite confusing as one might expect that `ng server` will trigger a server build

Closes: #10334
@angular-automatic-lock-bot
Copy link

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot bot locked and limited conversation to collaborators Sep 8, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
P3 An issue that is relevant to core functions, but does not impede progress. Important, but not urgent
Projects
None yet
Milestone
v6.0.0
3 participants
@hansl @Toxicable @alan-agius4