Fix and add browser support for tokenize function #100

fvictorio · 2020-01-23T19:31:53Z

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.

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 added 2 commits January 23, 2020 16:24

Fix tokenize function

9fe63fc

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 requested a review from federicobond January 23, 2020 19:32

fvictorio mentioned this pull request Jan 24, 2020

Fix and add browser support for tokenize function Consensys/solidity-parser-antlr#1

Merged

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Fix and add browser support for tokenize function #100

Fix and add browser support for tokenize function #100

fvictorio commented Jan 23, 2020

Fix and add browser support for tokenize function #100

Are you sure you want to change the base?

Fix and add browser support for tokenize function #100

Conversation

fvictorio commented Jan 23, 2020