Skip to content

Remove service worker impl generation from workbox plugin.#1152

Merged
LukeSheard merged 2 commits intomainfrom
feature/remove-service-worker
Dec 6, 2021
Merged

Remove service worker impl generation from workbox plugin.#1152
LukeSheard merged 2 commits intomainfrom
feature/remove-service-worker

Conversation

@LukeSheard
Copy link
Contributor

Remove support for the workbox plugin as we don't generate service workers by default and they are not important when it comes to our dynamic view loading architecture.

@changeset-bot
Copy link

changeset-bot bot commented Dec 6, 2021

🦋 Changeset detected

Latest commit: bf4e399

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
modular-scripts Minor

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@coveralls
Copy link
Collaborator

coveralls commented Dec 6, 2021

Coverage Status

Coverage increased (+0.009%) to 28.945% when pulling bf4e399 on feature/remove-service-worker into b4b438c on main.

Copy link
Contributor

@cristiano-belloni cristiano-belloni left a comment

Choose a reason for hiding this comment

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

Agree!

@LukeSheard LukeSheard merged commit 5c05494 into main Dec 6, 2021
@LukeSheard LukeSheard deleted the feature/remove-service-worker branch December 6, 2021 14:05
@github-actions github-actions bot mentioned this pull request Dec 6, 2021
LukeSheard added a commit that referenced this pull request Jan 24, 2022
* replace babel with esbuild for package compilation

* Add support for target as part of rollup transpilation

* fix package.json

* remove dist-cjs reference

* Create mean-bobcats-attend.md

* Bump tsconfig.json target to es2016 (#1154)

* Remove service worker impl generation from workbox plugin. (#1152)

* Simplify package.json scripts (#1155)

* simplify package.json scripts

* Update release.yml

* fix up failing tests

* Update output from builds

* Update mean-bobcats-attend.md
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