search

anza-xyz / move

Move compiler targeting llvm supported backends
https://discord.gg/wFgfjG9J
Apache License 2.0
108 stars 33 forks source link

NFC: Small corrections in move-mv-llvm-compiler::cli #318

Closed jcivlin closed 1 year ago

jcivlin commented 1 year ago

Small corrections, in particular, a passed function parameter should be an Option rather than Result.

Also VS Code config is removed.

ksolana commented 1 year ago

Also VS Code config is removed.

any reason we want to remove this?

dmakarov commented 1 year ago

https://github.com/solana-labs/move/actions/runs/6056987034/job/16437652599

jcivlin commented 1 year ago

Also VS Code config is removed.

any reason we want to remove this?

It should be local only. One that I use is very different and I need to maintain it manually because there is one in the repo.