Skip to content

Get Benchmarks Working#2

Open
confused-Techie wants to merge 5 commits into
masterfrom
update-benchmark
Open

Get Benchmarks Working#2
confused-Techie wants to merge 5 commits into
masterfrom
update-benchmark

Conversation

@confused-Techie
Copy link
Copy Markdown
Member

Was playing around with the benchmark file here, and ended up needing to update it.

This PR does the following:

  • Decafs benchmark.coffee => benchmark.js
  • Removes some Array.from() instances in the decaffed code
  • Wraps the whole benchmark in a self-invoking async function
  • Now that it's async it waits for onig.ready so that it can work with the new oniguruma
  • Fix the script called in the package.json

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