Skip to content

Fix solution for missing Mix manifest #99

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 2 commits into from
Jun 15, 2022
Merged

Conversation

jessarcher
Copy link
Contributor

This PR fixes the solution for a missing Mix manifest.

The exception message changed slightly a few months ago laravel/framework#42082

Verified

This commit was signed with the committer’s verified signature.
jessarcher Jess Archer
@jessarcher jessarcher changed the title Fix missing Mix manifest solution Fix solution for missing Mix manifest Jun 15, 2022
@jessarcher
Copy link
Contributor Author

I'm also wondering if there's a reason why this solution suggests npm ci over npm install? I'm assuming it's because people run into this more often in CI/prod than local development where that might be the preferred command?

@freekmurze
Copy link
Member

Thanks @jessarcher 🙌

I'm also wondering if there's a reason why this solution suggests npm ci over npm install? I'm assuming it's because people run into this more often in CI/prod than local development where that might be the preferred command?

Yes, indeed. In hindsight, it might be better to suggest npm instal, I'll take care of that

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
@freekmurze freekmurze merged commit 08380af into spatie:main Jun 15, 2022
@jessarcher jessarcher deleted the mix branch June 15, 2022 14:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants