aiken-lang / aiken

A modern smart contract platform for Cardano
https://aiken-lang.org
Apache License 2.0
479 stars 95 forks source link

I think there might be an issue with the vesting example. #1024

Open zootechdrum opened 1 month ago

zootechdrum commented 1 month ago

What Git revision are you using?

2.39.5

What operating system are you using, and which version?

Describe what the problem is?

I think there might be an issue with the vesting example. When I copy and paste the code on the first example and i run the command aiken check I get an error in the terminal stating that Blake2b_224 is an unknown module. I could be wrong, but I believe the modules should now be imported from crypto such as

use aiken/crypto.{Blake2b_224, Hash}
use aiken/crypto.{VerificationKey}

What should be the expected behavior?

I expected these modules to be imported without any issue.

Vesting example can be found here. https://aiken-lang.org/example--vesting

KtorZ commented 1 month ago

Hello, you probably missed the note at the top of the tutorial?

zootechdrum commented 1 month ago

Yes you are right @KtorZ . I completely missed it. I am wondering if the organization would be interested in an updated version of the example. I used the vesting example as a way to learn the language a little more and corrected any imports and syntax issues that the broken example had.

KtorZ commented 1 month ago

We are! The source for the example can be found in https://github.com/aiken-lang/site

Pull requests are welcome 🫡

stephanebachelier commented 1 month ago

@KtorZ Currently working on a Pull Request for vesting example.

KtorZ commented 1 month ago

Related to: https://github.com/aiken-lang/site/issues/85