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

Fix typos in Belt Module overview #2607

Merged
merged 1 commit into from Mar 13, 2018
Merged

Fix typos in Belt Module overview #2607

merged 1 commit into from Mar 13, 2018

Conversation

maheshsundaram
Copy link
Contributor

Hi there! I found a few small typos in the documentation for the Belt module overview. I read through the contribution guidelines, but if I missed anything, I'd be glad to know.

@bobzhang
Copy link
Member

hi @jasonmendes , thanks for your interest in contribution. The original source of documentation is here: https://github.com/BuckleScript/bucklescript/blob/master/jscomp/others/belt.ml#L26
when you have time, would you update the changes there?

@maheshsundaram
Copy link
Contributor Author

Hey @bobzhang Ah, sorry I missed that, and thank you for pointing that out! 🙇‍♂️ Yes, I will amend the changes to be there.

chenglou added a commit to chenglou/rescript-compiler that referenced this pull request Mar 13, 2018
@chenglou chenglou mentioned this pull request Mar 13, 2018
@chenglou
Copy link
Member

@jasonmendes Thanks for the PR! I'll spare you a little trouble and to the changes to the source in #2615.

Thanks again =)

@chenglou chenglou merged commit abc7edf into rescript-lang:master Mar 13, 2018
chenglou added a commit that referenced this pull request Mar 13, 2018
@maheshsundaram maheshsundaram deleted the belt-docs-typos branch March 13, 2018 15:16
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.

None yet

3 participants