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

Add native ESM test running via Snowpack to app-template-react #1197

Closed
wants to merge 64 commits into from

Conversation

FredKSchott
Copy link
Owner

@FredKSchott FredKSchott commented Oct 3, 2020

Depended on #1196, #1195 #1203 and #1193. (all merged)
Resolves #1036

Changes

  • 300 dependencies removed! (down from 954 -> 662)!
  • No more jest means no more second, jest-specific build pipeline!
  • Replaced with @web/test-runner, which uses Snowpack's internal build pipeline.
  • Testing through Snowpack = now possible for a larger Snowpack CDN integration

What is in this PR?

  • A new Snowpack plugin for @web/test-runner
  • Moving app-template-react from jest -> @web/test-runner
  • Added a server.shutdown() method to the Snowpack JS API

Testing

  • Added template tests to our current test suite.

Docs

  • TODO

@FredKSchott FredKSchott requested a review from a team as a code owner October 3, 2020 06:21
@vercel
Copy link

vercel bot commented Oct 3, 2020

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/pikapkg/snowpack/ch9o4c1r9
✅ Preview: https://snowpack-git-wip-web-test-runner.pikapkg.vercel.app

FredKSchott and others added 21 commits October 3, 2020 20:31
…es (#1198)

* add homepage and repository for published package.json files

* add homepage to babel-plugin-package-import

* add repository.directory setting

Co-authored-by: Patrick Miller <>
 - @snowpack/app-scripts-lit-element@1.8.6
 - @snowpack/app-scripts-preact@1.9.2
 - @snowpack/app-scripts-react@1.12.1
 - @snowpack/app-scripts-svelte@1.8.6
 - @snowpack/app-scripts-vue@1.8.7
 - @snowpack/app-template-11ty@1.8.2
 - @snowpack/app-template-blank-typescript@1.9.2
 - @snowpack/app-template-blank@1.6.14
 - @snowpack/app-template-lit-element-typescript@1.10.1
 - @snowpack/app-template-lit-element@1.9.1
 - @snowpack/app-template-preact@1.10.1
 - @snowpack/app-template-react-typescript@1.12.1
 - @snowpack/app-template-react@1.10.1
 - @snowpack/app-template-svelte-typescript@1.10.1
 - @snowpack/app-template-svelte@1.9.1
 - @snowpack/app-template-vue-typescript@1.10.1
 - @snowpack/app-template-vue@1.10.1
 - create-snowpack-app@1.8.3
 - esinstall@0.3.1
 - @snowpack/babel-plugin-package-import@1.1.4
 - @snowpack/plugin-babel@2.1.2
 - @snowpack/plugin-build-script@2.0.8
 - @snowpack/plugin-dotenv@2.0.3
 - @snowpack/plugin-optimize@0.2.3
 - @snowpack/plugin-parcel@2.0.5
 - @snowpack/plugin-postcss@1.0.5
 - @snowpack/plugin-react-refresh@2.3.1
 - @snowpack/plugin-run-script@2.1.4
 - @snowpack/plugin-svelte@2.1.1
 - @snowpack/plugin-vue@2.2.2
 - @snowpack/plugin-webpack@2.0.14
 - snowpack@2.13.2
Co-authored-by: Marko Heikkilä <marko.heikkila@nortal.com>
* chore: update tailwind import path

Otherwise Tailwinds purging and plugins are not working.

* Revert "chore: update tailwind import path"

This reverts commit e4ae781.

* chore: update using tailwind with postcss docs

* Update 08-guides.md

Co-authored-by: Fred K. Schott <fkschott@gmail.com>
* add strict tsconfig lint step

* add sep lint workflow

* add sep lint workflow

* add sep lint workflow

* add sep lint workflow

* Update lint.yml
* add typescript plugin, add tests

* update docs and templates

* Update plugin.js
g-plane and others added 3 commits October 6, 2020 20:46
…it found (#1233)

* writing the first docs test and fixing the errors it found

* Creating a GitHub Action

* Update docs.yml

* fixes to the test runner and actions

* removing jest from docs
Co-authored-by: Brian Jacobs <brian.jacobs@natgeo.com>
drwpow and others added 17 commits October 7, 2020 14:19
Webpack's magic comments must be extracted from dynamic imports before the spec is passed to `matchDynamicImportValue`. If a match is found for these types of comments, they are appended to the rewritten import.
* add plugin-sass with new onChange/markChanged interface

* add tests

* update docs

* update yarn lockfile

* update ts

* update snapshots

* add back missing snapshot tests

* update add true .sass support
* adding svelte-hmr

* make svelte-hmr a normal dep of the plugin, & lazy load svelte-hmr

* plugin svelte: document HMR options

* add bubbled esm-hmr property

* svelte plugin: upgrade svelte-hmr, & enable CSS only updates

* Update plugin.js

Co-authored-by: Fred K. Schott <fkschott@gmail.com>
…#1255)

* fix: do not assume a build script failed completely if there's stderr output

* refactor: address pr review comments
* Update 04-features.md

* Update 04-features.md

Co-authored-by: Fred K. Schott <fkschott@gmail.com>
* fix bad async sass test

* fix async sass tests

* fix docs test too
@FredKSchott FredKSchott changed the title [WIP] Add native ESM test running via Snowpack Add native ESM test running via Snowpack to app-template-react Oct 9, 2020
@FredKSchott
Copy link
Owner Author

All dependency PRs merged, closing in favor of a fresh PR: #1197

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.