TezosTaqueria / taqueria

*BETA* Taqueria provides a seamless development workflow to easily build, test and deploy your Tezos applications.
https://taqueria.io
Apache License 2.0
64 stars 20 forks source link

๐Ÿ› Bug โžพ if I do taq compile-all and not taq compile myfile.jsligo it create infite storageList files #1859

Closed zamrokk closed 1 year ago

zamrokk commented 1 year ago

๐Ÿšฅ Status (Internal Taqueria Team Use Only)

๐Ÿ†˜ What happened?

here my printscreen

image

See that every time I run the command it duplicates the file created for storage and parameter ?

๐Ÿ†˜ Steps to Reproduce?

TAQ_LIGO_IMAGE=ligolang/ligo:next taq compile-all

I use this version containing a ligo bugfix : npm i @taqueria/plugin-ligo@0.28.4-1852.e20decfd --registry https://npm.ecadinfra.com/

๐Ÿชต Relevant log output

No response

๐Ÿ˜ How impactful is this bug?

None

โฑ๏ธ Prevalance

No response

๐Ÿ’ป Operating System

None

๐Ÿ•ธ๏ธ System Architecture

None

๐ŸŒฎ Taqueria Version

specific look above

๐ŸŒฟ Node.js Version

No response

โ˜Ž๏ธ Contact Details

No response

Code of Conduct

mweichert commented 1 year ago

This needs to be tested against the work in #1864

mweichert commented 1 year ago

@zamrokk is this still occurring with v0.28.5?

zamrokk commented 1 year ago

on v0.28.5, taq compile-all does not work at all

image

mweichert commented 1 year ago

@zamrokk mind attaching one or more of your contracts here that I can use to test with? Thanks!

zamrokk commented 1 year ago

just this one : https://github.com/marigold-dev/training-dapp-1 if you do compile-all it does not work, if you compile pointing to the jsligo file, then is ok

TAQ_LIGO_IMAGE=ligolang/ligo:0.63.2 taq compile-all

TAQ_LIGO_IMAGE=ligolang/ligo:0.63.2 taq compile pokeGame.jsligo

mweichert commented 1 year ago

I tested against v0.28.5 and was able to reproduce. The problem is fixed in v0.28.6.