Skip to content
This repository has been archived by the owner on Feb 18, 2024. It is now read-only.

systemjs-reloader #896

Merged
merged 1 commit into from
May 15, 2016
Merged

systemjs-reloader #896

merged 1 commit into from
May 15, 2016

Conversation

npbenjohnson
Copy link
Contributor

New attempt at a reloader, adds clientside change debouncing, plugin loaded resource change detection, dynamic dependency parent tracking, recovery on import failure, static build for non-jspm users, Instance specific attachment, and hook for defining a custom module storage provider for unload/reload.

New attempt at a reloader, adds clientside change debouncing, plugin loaded resource change detection, dynamic dependency parent tracking, recovery on import failure, static build for non-jspm users, Instance specific attachment, and hook for defining a custom module storage provider for unload/reload.
@guybedford
Copy link
Member

Nice work! Out of interest, what issue in systemjs-hot-reloader inspired this?

@guybedford guybedford merged commit 718e44f into jspm:master May 15, 2016
@npbenjohnson
Copy link
Contributor Author

I really just wanted to fix reload of plugin loaded content, and allow for marking parents of dynamically loaded resources to support hot reloading svgs. Since the actual core function isn't too complex it was easier to just write es6 classes with separate responsibilities than cover regressions against existing single file solution. It should also make forward migration easier because the SystemJs access is all encapsulated in one class.
I think it covers:
alexisvincent/systemjs-hot-reloader#14
alexisvincent/systemjs-hot-reloader#32
alexisvincent/systemjs-hot-reloader#36
alexisvincent/systemjs-hot-reloader#49
alexisvincent/systemjs-hot-reloader#50
alexisvincent/systemjs-hot-reloader#74

@guybedford
Copy link
Member

Thanks, plugin support is a huge plus definitely!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants