Skip to content

feat(@schematics/angular): Added --project-root option to the library schematic#23994

Merged
clydin merged 1 commit into
angular:mainfrom
DariuszOstolski:main
Oct 7, 2022
Merged

feat(@schematics/angular): Added --project-root option to the library schematic#23994
clydin merged 1 commit into
angular:mainfrom
DariuszOstolski:main

Conversation

@DariuszOstolski
Copy link
Copy Markdown
Contributor

…hematics

PR Checklist

Please check to confirm your PR fulfills the following requirements:

PR Type

What kind of change does this PR introduce?

  • Bugfix
  • Feature
  • Code style update (formatting, local variables)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • CI related changes
  • Documentation content changes
  • Other... Please describe:

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?

  • Yes
  • No

Other information

@angular-robot angular-robot Bot added the feature Label used to distinguish feature request from other issues label Oct 1, 2022
@DariuszOstolski
Copy link
Copy Markdown
Contributor Author

Frankly speaking I have no idea why e2e windows test fails with the following message:

Running test "tests/build/bundle-budgets.ts" (2 of 14 [1:16] (18/214))...
==========================================================================================
Running `ng "build"` [silent]...
CWD: X:\ramdisk\e2e-main\e2e-test\test-project
==========================================================================================
Running `ng "build"` [silent]...
CWD: X:\ramdisk\e2e-main\e2e-test\test-project
Test Process error Error: Process exit error - "cmd.exe /c ng build": 134...

STDOUT:


STDERR:
FATAL ERROR: v8::FromJust Maybe value is Nothing.
 1: 00007FF7D48DFD2F v8::internal::CodeObjectRegistry::~CodeObjectRegistry+112495
 2: 00007FF7D486EEF6 v8::internal::wasm::WasmCode::safepoint_table_offset+65526
 3: 00007FF7D486FDAD node::OnFatalError+301
 4: 00007FF7D51A13A5 v8::V8::FromJustIsNothing+53
 5: 00007FF7D478CA4A RSA_meth_get_flags+22266
 6: 00007FF7D478C8ED RSA_meth_get_flags+21917
 7: 00007FF7D4786C10 node::SetTracingController+52592
 8: 00007FF7D4933A32 uv_pipe_pending_type+4242
 9: 00007FF7D493E4D0 uv_loop_init+1024
10: 00007FF7D493E77A uv_run+202
11: 00007FF7D490D885 node::SpinEventLoop+309
12: 00007FF7D47A8570 v8::internal::wasm::SignatureMap::Freeze+35776
13: 00007FF7D47A3BC8 v8::internal::wasm::SignatureMap::Freeze+16920
14: 00007FF7D492EF5D uv_poll_stop+557
15: 00007FF7D57624B0 v8::internal::compiler::RepresentationChanger::Uint32OverflowOperatorFor+146416
16: 00007FFCD2E27974 BaseThreadInitThunk+20
17: 00007FFCD322A2F1 RtlUserThreadStart+33


    at _exec (C:\Users\circleci\ng\tests\legacy-cli\e2e\utils\process.ts:77:19)
    at silentNg (C:\Users\circleci\ng\tests\legacy-cli\e2e\utils\process.ts:266:12)
    at ng (C:\Users\circleci\ng\tests\legacy-cli\e2e\utils\process.ts:254:16)
    at default_1 (C:\Users\circleci\ng\tests\legacy-cli\e2e\tests\build\bundle-budgets.ts:30:47)
    at async C:\Users\circleci\ng\tests\legacy-cli\e2e\utils\test_process.ts:15:9


Test "tests/build/bundle-budgets" failed...
Process error - "C:\Users\circleci\ng\tests\legacy-cli\e2e\utils\run_test_process,C:/Users/circleci/ng/tests/legacy-cli/e2e/tests/build/bundle-budgets



Exited with code exit status 1

@clydin clydin requested a review from alan-agius4 October 3, 2022 13:11
@alan-agius4 alan-agius4 added the action: review The PR is still awaiting reviews from at least one requested reviewer label Oct 4, 2022
Copy link
Copy Markdown
Collaborator

@alan-agius4 alan-agius4 left a 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.

Comment thread packages/schematics/angular/library/schema.json Outdated
Comment thread packages/schematics/angular/library/schema.json Outdated
Comment thread packages/schematics/angular/library/schema.json Outdated
@alan-agius4 alan-agius4 added action: cleanup The PR is in need of cleanup, either due to needing a rebase or in response to comments from reviews and removed action: review The PR is still awaiting reviews from at least one requested reviewer labels Oct 4, 2022
@DariuszOstolski
Copy link
Copy Markdown
Contributor Author

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,
Dariusz

@alan-agius4
Copy link
Copy Markdown
Collaborator

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, Dariusz

I rebased the PR for you.

Comment thread packages/schematics/angular/library/index.ts Outdated
Copy link
Copy Markdown
Collaborator

@alan-agius4 alan-agius4 left a 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.

@alan-agius4 alan-agius4 added the target: minor This PR is targeted for the next minor release label Oct 6, 2022
@angular-robot angular-robot Bot requested a review from alan-agius4 October 6, 2022 18:46
@alan-agius4 alan-agius4 added action: merge The PR is ready for merge by the caretaker and removed action: cleanup The PR is in need of cleanup, either due to needing a rebase or in response to comments from reviews labels Oct 6, 2022
Copy link
Copy Markdown
Collaborator

@alan-agius4 alan-agius4 left a 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.

@alan-agius4 alan-agius4 changed the title feat(@schematics/angular): Added --directory option to the library sc… feat(@schematics/angular): Added --project-root option to the library schematic Oct 6, 2022
Comment thread packages/schematics/angular/library/index.ts Outdated
@alan-agius4 alan-agius4 added action: merge The PR is ready for merge by the caretaker and removed action: merge The PR is ready for merge by the caretaker labels Oct 6, 2022
… schematics

With this change user is able to specify different project root for libraries than default one.

Closes: #11927
alan-agius4 added a commit to alan-agius4/angular-cli that referenced this pull request Oct 7, 2022
This option is useful to create an application outside of the `newProjectRoot`.

Related to angular#23994
@alan-agius4 alan-agius4 added this to the v15 Feature Freeze milestone Oct 7, 2022
alan-agius4 added a commit to alan-agius4/angular-cli that referenced this pull request Oct 7, 2022
… application schematic

This option is useful to create an application outside of the `newProjectRoot`.

Related to angular#23994
@clydin clydin merged commit d8bff4f into angular:main Oct 7, 2022
clydin pushed a commit that referenced this pull request Oct 7, 2022
… application schematic

This option is useful to create an application outside of the `newProjectRoot`.

Related to #23994
@angular-automatic-lock-bot
Copy link
Copy Markdown

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 Nov 7, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

action: merge The PR is ready for merge by the caretaker feature Label used to distinguish feature request from other issues target: minor This PR is targeted for the next minor release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants