Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ligo_v1.6.0 #46

Merged
merged 7 commits into from
Nov 14, 2024
Merged

ligo_v1.6.0 #46

merged 7 commits into from
Nov 14, 2024

Conversation

frankhillard
Copy link
Contributor

@frankhillard frankhillard commented Jun 13, 2024

Support of ligo compiler v1.6.0
Use Tezos.Next and Test.Next for future migration.

There is a warning left

File "lib/fa2.1/entrypoints/lambda_export.jsligo", line 120, characters 17-38:
119 |          let combinedOps = list([...ops, ...ticketOps]);
120 |          return [list([ticket, ...lt]), updatedLedger, updatedStorage, combinedOps];
121 |       },
:
Warning: variable cannot be used more than once.

@frankhillard frankhillard marked this pull request as ready for review July 19, 2024 13:36
@rinderknecht
Copy link

LGTM

@ziwot
Copy link
Contributor

ziwot commented Nov 12, 2024

Hello ! Thank you for this MR, why no merge yet ?
@frankhillard I made an MR on your MR frankhillard#1

note: lib version should maybe be bumped to 1.5.0 instead of 1.4.3

@ziwot
Copy link
Contributor

ziwot commented Nov 14, 2024

Hi, is this the new home of this repo ? https://gitlab.com/nomadic-labs-free-resources/fa2.1/ligo-fa2.1
What about packages ? (I mean, which one should a user use/study, which one will be packaged, will packages exist in the future ?)
Don't forget your users please

@EduardoRFS
Copy link
Contributor

@WalterProof sorry for the delay, @rinderknecht and I were looking on this again, I made some small changes but I will not be moving to 1.7.0 yet due to the "Method Not Allowed" issue, which may break some automations.

ligo.json Outdated Show resolved Hide resolved
@EduardoRFS EduardoRFS merged commit d0d2925 into ligolang:main Nov 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants