-
Notifications
You must be signed in to change notification settings - Fork 92
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
Jazz up the build system #240
Comments
All sound good to me! |
Hi @bennypowers are you working on 2) in your list above -- new test library? Or is someone else? Looking at fixing up some of the TypeScript nuances, including the component() HTMLElement issue , I think updating test files to TS would be a good thing to do first. I'm happy to work on that, or pick up anything you have in progress. |
Go for it. ping me here or on "Lit & Friends" slack if you need help |
@bennypowers (and @matthewp -- would love your input too if you have time!) How does this initial direction for TypeScript tests look? As mentioned to Benny on Slack, trying for a light touch initially. (Although I did add a test for attributes being undefined when not present to highlight the issue -- it was unexpected) |
Looks really good, and it was so considerate of you to prepare your comment with those easy-to-understand links, thank you! |
Looks great to me |
This issue proposes to:
@web/test-runner
2.1. use
esbuildPlugin
in wtr to run tests buildless and concurrently from ts sources2.2. add
"test"
npm script to run wtr@web/dev-server
to run the examples dir (maybe withnpm start
)make
with rollupmaster
->main
The text was updated successfully, but these errors were encountered: