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.

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

feat(@angular/cli): add ng4 option to ng new #4507

Merged
merged 2 commits into from
Feb 9, 2017
Merged

Conversation

hansl
Copy link
Contributor

@hansl hansl commented Feb 7, 2017

This allows to create a project with Angular 4 in the template.
Also fixes a few issues with ng4.

@hansl hansl force-pushed the ng4 branch 3 times, most recently from 2851b8f to b0afe31 Compare February 8, 2017 03:06
Copy link
Contributor

@filipesilva filipesilva left a comment

Choose a reason for hiding this comment

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

LGTM but have a look at the node7 CI. It failed with a weird missing chunk expectation and I restarted it.

@@ -24,7 +24,7 @@ export default function() {
} else {
// Otherwise create a project from scratch.
createProject = Promise.resolve()
.then(() => ng('new', 'test-project', '--skip-npm'))
.then(() => ng('new', 'test-project', '--skip-npm', argv['ng4'] ? '--ng4' : '--'))
Copy link
Contributor

Choose a reason for hiding this comment

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

Nit: '--' -> ''

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done.

The xi18n command was loading everything which was slowing down starting times.
@hansl hansl force-pushed the ng4 branch 2 times, most recently from 3e92bd1 to d162659 Compare February 8, 2017 19:51
This allows to create a project with Angular 4 in the template.
Also fixes a few issues with ng4.
@hansl hansl merged commit c096afb into angular:master Feb 9, 2017
@hansl hansl deleted the ng4 branch February 9, 2017 00:41
@Splaktar
Copy link
Member

Should this be ngnext instead of being hardcoded to v4.x (ng4)? I imagine that the need for generating projects with the latest (alpha/beta) Angular will continue to exist after ng4 is GA (i.e. ng5-alpha).

@fmalcher
Copy link
Contributor

fmalcher commented Feb 24, 2017

Are there any plans to remove the ng4 option again once Angular 4 is released – or will it stay here forever?
@hansl @filipesilva

@filipesilva
Copy link
Contributor

@Splaktar @fmalcher we're going to do something along those lines, yes.

asnowwolf pushed a commit to asnowwolf/angular-cli that referenced this pull request Apr 12, 2017
This allows to create a project with Angular 4 in the template.
Also fixes a few issues with ng4.
@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 11, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants