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

Deno peer #2

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Deno peer #2

wants to merge 5 commits into from

Conversation

tylerchilds
Copy link

Stable work in progress-- ish.

All the commands in the readme start, but deno.js a direct port of demo.js, with only switching require to import and then running with deno.

This command fails with

error: Error in [email protected] parsing version requirement for dependency: web-streams-polyfill@git://github.com/gwicke/web-streams-polyfill#spec_performance_improvements

Invalid npm version requirement. Unexpected character.
  git://github.com/gwicke/web-streams-polyfill#spec_performanc
  ~

Caused by:
    0: Invalid npm version requirement. Unexpected character.
         git://github.com/gwicke/web-streams-polyfill#spec_performanc
         ~
    1: Unexpected character.
         git://github.com/gwicke/web-streams-polyfill#spec_performanc
         ~

pending a fix to the web-streams-polyfill dependency, we'll likely face issues with the require statements in index.js and below.

@tylerchilds
Copy link
Author

i’ve resolved the require issues in index.js as a fork in braidmail.js

we’re back the the web-streams-polyfill error. pending that, this pr is pretty close.

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.

1 participant