Stencil: Stencil 3 support? #11
Comments
|
Hi @sarahquigley I did a quick experiment with Stencil 3. Seems like the e2e tests are running with the update to Stencil 3 with some slightly changes. The biggest problem here is still that the user have to downgrade to jest 27. I don't do that automatically cause it could break all other Nx packages cause for Nx Jest 28 is the default. Hopefully this will be fixed some day for Stencil |
|
@DominikPieper Yes, agree lack of Jest 28 support is a problem. Hope that will be resolved on Stencil's end soon. We're running on Jest 27 for now. Re: E2E tests, as I said above, when we upgrade out project to Stencil 3, we have issues issues running the E2E tests. The E2E tests run perfectly with Stencil 2, but as soon as we upgrade, we encounter the following error: It seems running our e2e tests no longer respect Do you have a work around for this issue with Stencil 3? |
|
Could you maybe create an minimal example of the error or is the project on github? I could than do some experiments around that specific error |
|
The project is private unfortunately, but I'll create minimal example - and get that to you ASAP. @DominikPieper Appreciate the help, thank you! |
|
@sarahquigley We have upgraded our project to Stencil 3 and we were able to keep e2e running properly. Indeed we need to override some dependencies in the package.json. You can check the repo here, hope it helps. cc. @DominikPieper |
|
FYI: with little bit patience and luck we get the support in Stencil 4.x |
|
Hello, we are currently using nx 16.5.0, @nxext/stencil 16.5.0, stencil 4. Everything works fine including tests. |
@erichstark do you have a repo we can check, if you dont mind, or maybe sharing you package.json and NX config. I would like to know if you have unit and e2e tests within Stencil and not through other testing libraries. |
|
I managed to update my Nx monorepo to the latest Nx version ( I'm posting here the relevant information for those who will like to update to Nx 16 while keeping unit tests and e2e tests working within Stencil Jest and Puppeteer. // package.json
devDependencies: {
...
"@nx/angular": "16.6.0",
"@nx/devkit": "16.6.0",
"@nx/eslint-plugin": "16.6.0",
"@nx/jest": "16.6.0",
"@nx/js": "16.6.0",
"@nx/linter": "16.6.0",
"@nx/plugin": "16.6.0",
"@nx/react": "16.6.0",
"@nx/rollup": "16.6.0",
"@nx/storybook": "16.6.0",
"@nx/web": "16.6.0",
"@nx/workspace": "16.6.0",
"@nxext/stencil": "16.3.0",
...
"@types/jest": "27.5.1",
...
"@stencil/angular-output-target": "0.7.1",
"@stencil/core": "4.0.2",
"@stencil/react-output-target": "0.5.3",
"@stencil/sass": "3.0.5",
...
"babel-jest": "27.5.1",
"jest": "27.5.1",
"jest-environment-jsdom": "27.5.1",
"ts-jest": "27.1.5",
...
"nx": "16.6.0",
"nx-cloud": "16.2.0",
"nx-stylelint": "15.0.0",
},
"overrides": {
"@jest/environment": "$jest",
"@nx/jest": {
"jest": "$jest"
},
"@nxext/stencil": {
"@nx/angular": "$@nx/angular",
"@nx/jest": "$@nx/jest",
"@nx/linter": "$@nx/linter",
"@nx/react": "$@nx/react",
"@nx/storybook": "$@nx/storybook",
"@nx/workspace": "$@nx/workspace",
"@stencil/core": "$@stencil/core"
},
"@typescript-eslint/eslint-plugin": "$@typescript-eslint/eslint-plugin",
"@typescript-eslint/parser": "$@typescript-eslint/parser",
"jest-config": "$jest",
},Nx report: > NX Report complete - copy this into the issue template
Node : 18.16.1
OS : darwin-arm64
npm : 9.8.0
nx : 16.6.0
@nx/js : 16.6.0
@nx/jest : 16.6.0
@nx/linter : 16.6.0
@nx/workspace : 16.6.0
@nx/angular : 16.6.0
@nx/cypress : 16.3.0
@nx/devkit : 16.6.0
@nrwl/devkit : 15.9.4
@nx/eslint-plugin : 16.6.0
@nx/plugin : 16.6.0
@nx/react : 16.6.0
@nx/rollup : 16.6.0
@nx/storybook : 16.6.0
@nrwl/tao : 16.6.0
@nx/web : 16.6.0
@nx/webpack : 16.6.0
nx-cloud : 16.2.0
typescript : 4.8.4
---------------------------------------
Community plugins:
@jscutlery/semver : 2.29.3
@nxext/stencil : 16.3.0
ngx-deploy-npm : 6.0.0
nx-stylelint : 15.0.0
---------------------------------------
Local workspace plugins:
...
---------------------------------------
The following packages should match the installed version of nx
- @nx/[email protected]
- @nrwl/[email protected]
- @nrwl/[email protected]
To fix this, run `nx migrate [email protected]`More details can be seen here in this Pull Request PS: I also have Storybook v7.1.0 working with this setup |
|
Here is mine package.json
|
Is your feature request related to a problem? Please describe.
We have Stencil + NX project that uses @nxext/stencil. We want to upgrade to Stencil 3, but we noted this @next/stencil doesn't yet support Stencil 3. Are there plans to add support for Stencil 3?
Describe the solution you'd like
Stencil 3 support. :)
Describe alternatives you've considered
We have experimented with upgrading to Stencil 3 using the current version of @nxext/stencil - so far (though we're still testing the upgrade) the main issue we've encountered is that we are unable to run our e2e tests. Upgrading reintroduces this issue: nxext/nx-extensions#310
We are eager to upgrade to Stencil 3 ASAP, so if an upgrade isn't in the works here soon, we'll have to consider whether we can patch this issue ourselves or will need to move away from this plugin.
The text was updated successfully, but these errors were encountered: