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

Update community polyfills and tooling list #231

Merged
merged 2 commits into from
Oct 23, 2020
Merged

Conversation

domenic
Copy link
Collaborator

@domenic domenic commented Oct 22, 2020

See #146.

README.md Outdated
* [@import-maps/resolve](https://www.npmjs.com/package/@import-maps/resolve) resolves a specifier relative to an import map.
* [@jsenv/node-module-import-map](https://www.npmjs.com/package/@jsenv/node-module-import-map) generates an import map from your `package.json` and `node_modules/` directories.
* [Snowpack](https://www.snowpack.dev) generates an import map when it installs your npm packages to run on the web.
* [Built-in Module Demo (with Rollup)](https://glitch.com/edit/#!/rollup-built-in-modules) contains code for a [Rollup](https://rollupjs.org/) plugin that generates an import map, focused on built-in module polyfills.
* [@jsenv/importmap-eslint-resolver](https://www.npmjs.com/package/@jsenv/importmap-eslint-resolver) enables import map resolution in [ESLint](https://eslint.org/)
* [Deno](https://github.com/denoland/deno) is a JavaScript/TypeScript runtime with [built-in support for import maps](https://deno.land/manual/linking_to_external_code/import_maps).
* [es-dev-server](https://www.npmjs.com/package/es-dev-server) allows using import maps during development, including polyfills.
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
* [es-dev-server](https://www.npmjs.com/package/es-dev-server) allows using import maps during development, including polyfills.
* [web-dev-server](https://www.npmjs.com/package/@web/dev-server) allows using import maps during development, including polyfills.
* [web-test-runner](https://www.npmjs.com/package/@web/test-runner) while testing you can mock external es modules via import maps

for reference some docs for Web Test Runner regarding mocking
https://modern-web.dev/docs/test-runner/writing-tests/mocking/

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

To clarify: web-dev-server is the successor of es-dev-server :) We have a dedicated import map plugin for both the dev server and the test runner: https://modern-web.dev/docs/dev-server/plugins/import-maps/

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! I'll consolidate these into a link to https://www.npmjs.com/package/@web/dev-server-import-maps

* [@import-maps/resolve](https://www.npmjs.com/package/@import-maps/resolve) resolves a specifier relative to an import map.
* [@jsenv/node-module-import-map](https://www.npmjs.com/package/@jsenv/node-module-import-map) generates an import map from your `package.json` and `node_modules/` directories.
* [Snowpack](https://www.snowpack.dev) generates an import map when it installs your npm packages to run on the web.
* [Built-in Module Demo (with Rollup)](https://glitch.com/edit/#!/rollup-built-in-modules) contains code for a [Rollup](https://rollupjs.org/) plugin that generates an import map, focused on built-in module polyfills.
* [@jsenv/importmap-eslint-resolver](https://www.npmjs.com/package/@jsenv/importmap-eslint-resolver) enables import map resolution in [ESLint](https://eslint.org/)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good for me.

For the record I have updated @jsenv/importmap-eslint-resolver readme to make it better, thanks for the link ;)

@domenic domenic merged commit 535b545 into master Oct 23, 2020
@domenic domenic deleted the more-community-stuff branch January 8, 2021 20:22
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.

4 participants