-
Notifications
You must be signed in to change notification settings - Fork 2
Upgrade Luerl #3
Copy link
Copy link
Open
Labels
good first issueGood for newcomersGood for newcomershelp wantedExtra attention is neededExtra attention is neededupgradecode that exists but needs to upgrade dependencies and adapt to themcode that exists but needs to upgrade dependencies and adapt to them
Description
Looks like github.com/luerl evolved at some point including by default Elixir modules, but also changing the name of the functions. Not sure if the internal behaviour could be affected so, in this ticket I guess could be great idea:
- Upgrade luerl
- Change the use of
:luerlmodule to useLuerl - Check the use of the functions because
:luerl.set_table/3and:luerl.eval/2doesn't exist anymore. I guess they were renamed to:luerl.set_table_keys/3so we could useLuerl.set_table_keys/3and:luerl.eval/2was renamed to:luerl.do/2so we could useLuerl.do/2. - Make tests!
Yes, I know, there are no enough tests, but it's something we have to change :-)
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
good first issueGood for newcomersGood for newcomershelp wantedExtra attention is neededExtra attention is neededupgradecode that exists but needs to upgrade dependencies and adapt to themcode that exists but needs to upgrade dependencies and adapt to them