Skip to content
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

Configures DPUL to use Lando's Redis #943

Merged
merged 3 commits into from
Sep 15, 2021
Merged

Configures DPUL to use Lando's Redis #943

merged 3 commits into from
Sep 15, 2021

Conversation

sdellis
Copy link
Member

@sdellis sdellis commented Sep 15, 2021

Fixes #941

Also updates webpack in a separate commit. Resolves errors that occur when running webpack locally.

@tpendragon
Copy link
Contributor

The error this fixes in webpack:

dpul main % bin/webpack-dev-server
The command moved into a separate package: @webpack-cli/serve
Would you like to install serve? (That will run yarn add -D @webpack-cli/serve) (yes/NO) : yes
yarn add v1.22.11
[1/4] 🔍  Resolving packages...
[2/4] 🚚  Fetching packages...
[3/4] 🔗  Linking dependencies...
warning " > @webpack-cli/serve@1.5.2" has incorrect peer dependency "webpack-cli@4.x.x".
[4/4] 🔨  Building fresh packages...
success Saved lockfile.
success Saved 1 new dependency.
info Direct dependencies
└─ @webpack-cli/serve@1.5.2
info All dependencies
└─ @webpack-cli/serve@1.5.2
✨  Done in 5.13s.
TypeError: Class constructor ServeCommand cannot be invoked without 'new'
    at runWhenInstalled (/Users/tpendragon/Projects/dpul/node_modules/webpack-cli/bin/utils/prompt-command.js:46:9)
    at /Users/tpendragon/Projects/dpul/node_modules/webpack-cli/bin/utils/prompt-command.js:124:15
    at processTicksAndRejections (internal/process/task_queues.js:97:5)

@tpendragon tpendragon merged commit 09547ac into main Sep 15, 2021
@tpendragon tpendragon deleted the issue-941 branch September 15, 2021 18:19
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.

Configure DPUL to use Lando's Redis
2 participants