A modern programming language that makes developers happier.

kotlinlang

also it seems the docs here `<https://github.com/arrow-kt/arrow/blob/main/CONTRIBUTING.md#steps>` are outdated.
```cd arrow-libs/core
./gradlew build```
doesn't work any more

Hey <@U0WTNS11Q>,

We also have <#C8UK6RTHU|arrow-contributors> for these kind of discussions.

Strange that the testsuite is not failing. That’s problematic :confused:

Yes, that markdown is indeed outdated. We’re currently in the progress of updating Dokka, and replacing Ank with Knit etc.

However IIRC the `buildDoc` or `runSite` command should still work. Is this failing?

Hey <@U0RM4EPC7> I realized that right after I posted here :sweat_smile:
I think `buildDoc` or `runSite` works but in root directory not in `arrow-libs` directory anymore.
I faced this error when trying to add a `filter` function to `Optional` in optics (as we discussed few weeks ago).

I created a pr for it <https://github.com/arrow-kt/arrow/pull/2588>. It is in draft, because I need to find out how to write tests for it, and it needs some more documentation (I probably just copy docs from Monocle :smile: )

is Ank getting deprecated in favor of Knit? :thinking_face:

<@UDAV1EN4W> yes. We’ve already replaced all the ank usage in Arrow with Knit

<@U0WTNS11Q> you can add test specific for `filter` here, <https://github.com/arrow-kt/arrow/blob/main/arrow-libs/optics/arrow-optics/src/commonTest/kotlin/arrow/optics/OptionalTest.kt>

If it passes the law, we should still test for specific behavior since the laws just check if it adheres to the generic behaviors of `Optional`.

Yes, it doesn’t work from the folder anymore since we’ve flattened the project for simpler maintenance.

I just added a test `Optional.listHead` to make sure `set` function work correctly: <https://github.com/arrow-kt/arrow/pull/2589>

Hey <@U0RM4EPC7>, when you have time can you please review my two PRs above. I'll update the first PR with some more document. But I don't not sure how to add test laws for it. It seems in the laws, we try to compose the function with another function and make sure the result satisfies all the laws?

I’ll try to make some time this week to properly look into it :+1:

Thanks for the awesome work <@U0WTNS11Q> :raised_hands: