Ethers.js: Dollar sign is valid identifier character but not accepted

[frangio]

Ethers.js doesn't accept function names with a dollar sign in them.

https://github.com/ethers-io/ethers.js/blob/4e6d121fb8aa7327290afab7653364be8ddd8d81/packages/abi/src.ts/fragments.ts#L1032

See the lexer definition in the Solidity repo:

An identifier in solidity has to start with a letter, a dollar-sign or an underscore and may additionally contain numbers after the first symbol.

[ricmoo]

Thanks. I’ll get to this along with another parser issue sometime this week.

[ricmoo]

This has been fixed in 5.3.1. Try it out and let me know if you have any issues.

Thanks! :)

[frangio]

@ricmoo Thank you! Seems to work.