Conversation
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the ✨ Finishing touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
UncleGrumpy
force-pushed
the
escritpize
branch
3 times, most recently
from
733718c to
f8e0690
Compare
This makes a few small adjustments to the escriptize provider as well as the tests, most of these are necessary changes. A few additional changes have heen made, such as adding the users $HOME/.local, to the default AtomVV install search paths, and supporting matching on the AtomVM elf executable, as well as the shell launcher script. Signed-off-by: Winford <winford@object.stream>
UncleGrumpy
force-pushed
the
escritpize
branch
from
f8e0690 to
6b652b4
Compare
|
I got the workflow updated to build and install the AtomVM runtime allowing the successful completion of the test. Link to the latest run: |
pguyot
force-pushed
the
w44/escriptize
branch
from
da31bd3 to
f50be09
Compare
pguyot
force-pushed
the
w44/escriptize
branch
from
f50be09 to
ff5c0e2
Compare
This makes a few small adjustments to the escriptize provider as well as the tests, most of these are necessary changes. A few additional changes have heen made, such as adding the users $HOME/.local, to the default AtomVV install search paths, and supporting matching on the AtomVM elf executable, as well as the shell launcher script. Signed-off-by: Winford <winford@object.stream>
2 participants
I think you will need to rebase for this to merge cleanly, but these changes will help us get atomvm#59 closer to the finish line. We still need to pull in a copy of AtomVM and atomvmlib from main to assemble the escript, we should probably put them in proper install locations.