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

#1168 replace autoDefineCustomElements with customElementsExportBehavior 'auto-define-custom-elements' #1169

Conversation

NickersWeb
Copy link
Contributor

Related issue: #1168

…om-elements autoDefineCustomElements true with customElementsExportBehavior 'auto-define-custom-elements', including tests (https://stenciljs.com/docs/custom-elements#customelementsexportbehavior)
Copy link

nx-cloud bot commented Jan 28, 2025

View your CI Pipeline Execution ↗ for commit d5760e3.

Command Status Duration Result
nx affected --target lint build test e2e ✅ Succeeded 33s View ↗
nx-cloud record -- pnpm exec nx format:check ✅ Succeeded 2s View ↗

☁️ Nx Cloud last updated this comment at 2025-01-29 13:47:55 UTC

@NickersWeb NickersWeb marked this pull request as ready for review January 28, 2025 09:30
@DominikPieper
Copy link
Member

@NickersWeb looks good, you just need to run the formatter. That's why the pipeline is failing

@edbzn
Copy link
Member

edbzn commented Jan 28, 2025

Commit message are not formatted correctly.

@NickersWeb
Copy link
Contributor Author

NickersWeb commented Jan 28, 2025

Commit message are not formatted correctly.

@edbzn I'm seeing no documentation in the markdown files; is this in reference to .husky and commit-msg? If so, please describe how I can resolve this issue.

@edbzn
Copy link
Member

edbzn commented Jan 28, 2025

We can do it ourself when squashing no worries

@DominikPieper
Copy link
Member

DominikPieper commented Jan 28, 2025

@NickersWeb for future commits, you can just do "pnpm commit". The workflow then will ask for the needed informations :-) there's a commitizen workflow

…get-dist-custom-elements-custom-elements-export-behavior
@DominikPieper DominikPieper merged commit 0b1fce3 into nxext:main Feb 21, 2025
7 checks passed
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.

3 participants