Skip to content

feat(@angular/cli): provide more detailed error for not found builder #22176

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

Merged
merged 1 commit into from
Nov 17, 2021

Conversation

clydin
Copy link
Member

@clydin clydin commented Nov 16, 2021

When a builder-based command is executed (build, serve, test, etc.) and the builder's node package cannot be found a more user-friendly error message is now displayed. In addition, when the builder's node package cannot be found, a check is performed to determine if the node packages for the workspace may have not been installed. Previously, a potentially long stacktrace was shown which did not provide much information regarding how to correct the issue.

Closes: #10536

@clydin clydin added the target: minor This PR is targeted for the next minor release label Nov 16, 2021
@google-cla google-cla bot added the cla: yes label Nov 16, 2021
When a builder-based command is executed (build, serve, test, etc.) and the builder's node package cannot be found a more user-friendly error message is now displayed. In addition, when the builder's node package cannot be found, a check is performed to determine if the node packages for the workspace may have not been installed. Previously, a potentially long stacktrace was shown which did not provide much information regarding how to correct the issue.

Closes: angular#10536
@clydin clydin force-pushed the better-error-missing-builder branch from 69b5bb0 to 9d558a3 Compare November 16, 2021 23:34
@alan-agius4 alan-agius4 added the action: merge The PR is ready for merge by the caretaker label Nov 17, 2021
@filipesilva filipesilva merged commit ecd9fb5 into angular:master Nov 17, 2021
@clydin clydin deleted the better-error-missing-builder branch November 17, 2021 16:00
@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 Dec 18, 2021
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 target: minor This PR is targeted for the next minor release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Better/nicer error message if a CLI user forgets to add packages
3 participants