Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Introduce prefetch chunks build #171

Merged
merged 3 commits into from
Apr 19, 2020
Merged

Introduce prefetch chunks build #171

merged 3 commits into from
Apr 19, 2020

Conversation

addyosmani
Copy link
Collaborator

This is a follow-up to #156 which adds support for prefetch chunks as a separate build. The goal of this build is to give developers who are building single-page applications to logic required to do so, without requiring all users to pay the extra byte-costs.

addyosmani and others added 3 commits April 2, 2020 18:09
* [fix] unit tests for prefetchchunks

* [chore] cleaned up console logs

* [infra] default accessor argument added
* [fix] mistyped expected value

* [chore] debugging prefetch chunks test

* [chore] debugging with normal JS functions

* [chore] debugging by adding a real css file

* [chore] debugging

* [chore] debugging

* [chore] debugging

* [chore] debugging

* [chore] debugging

* [chore] debugging

* [chore] debugging

* [chore] debugging

* [chore] debugging tests

* [chore] debugging tests

* [chore] cleaned up
@anton-karlovskiy
Copy link
Contributor

@addyosmani cc @demianrenzulli

LGTM!

@demianrenzulli
Copy link
Contributor

Just went through the code and couldn't find any issues there, so it LGTM too 👍

@addyosmani
Copy link
Collaborator Author

Thank you, @anton-karlovskiy and @demianrenzulli!

@addyosmani addyosmani merged commit 301aedb into master Apr 19, 2020
@addyosmani addyosmani deleted the prefetchchunks-plugin branch April 19, 2020 01:43
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants