-
Notifications
You must be signed in to change notification settings - Fork 12k
feat(@schematics/angular): Added --project-root option to the library schematic #23994
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
Conversation
Frankly speaking I have no idea why e2e windows test fails with the following message:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you also add Closes: #11927
as part of the commit message footer as this issue fixes #11927 Thanks.
Hi, I'm not sure why tests failed on CircleCi, is there something I have to do on my branch? The issue seems to be with node version, but this is out of scope of this pull request. BR, |
I rebased the PR for you. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
One NIT, otherwise LGTM.
Please squash the commits.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks for your contribution.
… schematics With this change user is able to specify different project root for libraries than default one. Closes: #11927
This option is useful to create an application outside of the `newProjectRoot`. Related to angular#23994
… application schematic This option is useful to create an application outside of the `newProjectRoot`. Related to angular#23994
… application schematic This option is useful to create an application outside of the `newProjectRoot`. Related to #23994
This issue has been automatically locked due to inactivity. Read more about our automatic conversation locking policy. This action has been performed automatically by a bot. |
…hematics
PR Checklist
Please check to confirm your PR fulfills the following requirements:
PR Type
What kind of change does this PR introduce?
What is the current behavior?
Issue Number: 11927
What is the new behavior?
User is able to specify subdirectory in which new library is created
Does this PR introduce a breaking change?
Other information