Skip to content
This repository was archived by the owner on Sep 27, 2020. It is now read-only.

Fix and add browser support for tokenize function #100

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

Fix and add browser support for tokenize function #100

wants to merge 2 commits into from

Conversation

@fvictorio
Copy link
Contributor

@fvictorio fvictorio commented Jan 23, 2020

This PR makes two changes:

  • Fix an issue with the tokenize function (the necessary files weren't being copied when the build was done)
  • Make the tokenize function work in the browser. Before this, the Solidity.tokens was read on runtime using the fs module. I added a babel plugin to inline it in the necessary module.

@fvictorio
Fix tokenize function
9fe63fc
@fvictorio
Inline Solidity.tokens contents when building the project
64c8d99 
The tokenize function uses `fs` to read Solidity.tokens on runtime. This
doesn't work in the browser. With this change, the parse and tokenize
functions can be used in the browser.
@fvictorio fvictorio mentioned this pull request Jan 24, 2020
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@federicobond federicobond Awaiting requested review from federicobond

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@fvictorio