[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: david30907d
To complete the pull request process, please assign pradithya
You can assign the PR to them by writing /assign @pradithya in a comment when ready.

The full list of commands accepted by this bot can be found here.

Hi @david30907d. Thanks for your PR.

I'm waiting for a gojek member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.

Once the patch is verified, the new status will be reflected by the ok-to-test label.

I understand the commands that are listed here.

Hi @david30907d

Thanks for the PR!

I am a little bit divided on whether to merge this. The markdown is still quite readable even with the Gitbook annotations. The advantage is that it's a lot more readable if you do use Gitbook.

Can you have a look at Feast Gitbook?

Hi @david30907d

Thanks for the PR!

I am a little bit divided on whether to merge this. The markdown is still quite readable even with the Gitbook annotations. The advantage is that it's a lot more readable if you do use Gitbook.

Can you have a look at Feast Gitbook?

Hi @woop , thanks for your reply, cause i didn't notice this GitBook page 😅
How about add a GitBook page for developer in case that they don't know Feast Gitbook ?

I've open a new PR for this #385 .