-
Notifications
You must be signed in to change notification settings - Fork 1
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
Investigate and potentially remove dependencies on browser-only APIs #9
Comments
@websdk/contributors is anyone using the |
I don' think so, but I'll check and get back to you |
Much obliged @aaronhaines! |
Ah yes, browser dependencies. Would be nice if there was an easy way to include/exclude platform specific extension like this. I just put everything in one file, because, well, files. |
Also, as I recall it made my life much harder when I was trying to get this to run in node, but I guess no one is doing that at the moment(?), but could well be valuable. |
I've got #10 running in node. Removing the dependency on |
From @mstade on September 18, 2015 13:37
Currently, nap depends on at least
window.document.matchesSelector
but possibly others as well. This dependency causes some issues in supporting execution in both server side and client side contexts, and should possibly be removed altogether. If it can't be removed, care must be taken not to include superfluous dependencies in client side contexts.Copied from original issue: sammyt/nap#21
The text was updated successfully, but these errors were encountered: