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

chore: rename main function on main page example #1097

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Unn4m3DD
Copy link

This is a tiny pr renaming the main function on the first usage example of the lib.
Even though it probably looks like it, this is not a troll hacktoberfest I want my free t-shirt kind of PR.
I just picked up ocaml and this was the first code example I got up and running. I was very confused by the fact that there was this variable body everywhere and couldn't quite figure out what it meant. Then it clicked that there are indeed 2 variables called body each of which has nothing to do with each other.
I believe many others will also start their ocaml journey by making some http requests and it would be great to have this example be more self explanatory.
This is even done later on this same readme under ## Basic server tutorial

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.

1 participant