Use Default NPM Package Manager of User if Lock File Exists In Base Path #1187
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR will fix errors that occur after initial Jetstream installation if a User is using an NPM package manager other than NPM itself
Many people initially start a fresh Laravel project by configuring their database and using their favorite package manager whether that be npm, yarn, or pnpm. Jetstream's Console InstallCommand.php file autoloads the bundling of assets using npm as the default package manager even while a user may be already using pnpm or yarn. If a user is already using pnpm or yarn they will receive an error in their terminal that looks similar to the following:
Even though this is an easy fix for the user by just finishing installation and migrating their tables, people who are unexperienced may not know what is going on or may think there is a problem to find and fix in their project...
A simple fix to this is just adding the following sections of code in the "InstallCommand.php" file. People like to use their default "favorite" package manager and I see no harm in allowing them to. For this reason alone I am submitting this PR...