qwik icon indicating copy to clipboard operation
qwik copied to clipboard

feat: add docker compose - STILL WIP

Open shairez opened this issue 3 years ago • 2 comments

This is not done yet, I'm running into an error

See my other comment below

What is it?

  • [x] Feature / enhancement
  • [ ] Bug
  • [ ] Docs / tests

Description

this will enable an easier way for contributors to test stuff locally without installing all of the depenedencies like rust etc.

see #794

Use cases and why

    1. One use case
    1. Another use case

Checklist:

  • [x] My code follows the developer guidelines of this project
  • [x] I have performed a self-review of my own code
  • [ ] I have made corresponding changes to the documentation
  • [ ] Added new tests to cover the fix / functionality

shairez avatar Jul 27 '22 20:07 shairez

Hey, I'm getting the following error inside of the docker container -

ReferenceError: fetch is not defined
    at Module.init (file:///opt/qwik/node_modules/@builder.io/qwik/bindings/qwik.wasm.mjs:421:9)
    at loadPlatformBinding (file:///opt/qwik/node_modules/@builder.io/qwik/optimizer.mjs:537:25)
    at async createOptimizer (file:///opt/qwik/node_modules/@builder.io/qwik/optimizer.mjs:567:85)
    at async Object.init (file:///opt/qwik/node_modules/@builder.io/qwik/optimizer.mjs:832:47)
    at async Object.config (file:///opt/qwik/node_modules/@builder.io/qwik/optimizer.mjs:1414:7)
    at async resolveConfig (file:///opt/qwik/node_modules/vite/dist/node/chunks/dep-1513d487.js:62385:25)
    at async doBuild (file:///opt/qwik/node_modules/vite/dist/node/chunks/dep-1513d487.js:43308:20)
    at async build (file:///opt/qwik/node_modules/vite/dist/node/chunks/dep-1513d487.js:43297:16)
    at async CAC.<anonymous> (file:///opt/qwik/node_modules/vite/dist/node/cli.js:747:9)

@manucorporat it doesn't seem like a windows issue, for some reason, fetch cannot be found where evet the wsamgets run 🤷‍♂️

shairez avatar Jul 27 '22 20:07 shairez

OK Build issues are solved

Still need to fix the running of the e2e tests (I think it's because the docker image is not ubuntu) I'll continue tomorow

shairez avatar Jul 27 '22 22:07 shairez

@shairez can we close this PR in favor of https://github.com/BuilderIO/qwik/pull/5237 ?

zanettin avatar Sep 30 '23 18:09 zanettin

Yes, thanks for the ping @zanettin !

shairez avatar Oct 05 '23 16:10 shairez