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

Remove unused dev dependency from build directory #27612

Closed
wants to merge 1 commit into from

Conversation

juliusknorr
Copy link
Member

puppeteer is no longer used anywhere so we can drop it

@LukasReschke
Copy link
Member

The node tests are failing now https://github.com/nextcloud/server/pull/27612/checks?check_run_id=2887817487

22 06 2021 18:22:49.805:ERROR [config]: Error in config file!
1285
  Error: Cannot find module 'puppeteer'
1286
Require stack:
1287
- /home/runner/work/server/server/tests/karma.config.js
1288
- /home/runner/work/server/server/build/lib/node_modules/karma/lib/config.js
1289
- /home/runner/work/server/server/build/lib/node_modules/karma/lib/server.js
1290
- /home/runner/work/server/server/build/lib/node_modules/karma/lib/cli.js
1291
- /home/runner/work/server/server/build/lib/node_modules/karma/bin/karma
1292

@juliusknorr
Copy link
Member Author

Oh then I indeed mixed things up there, i had puppeteer in mind as a leftover from the ui regression testing attempt, but seems past-me reintroduced it in c7a320d 🙈 Sorry for the noise, please move on.

@juliusknorr juliusknorr deleted the deps/build-unused branch June 22, 2021 19:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants