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

Clarify writing in 'Declarations' section #716

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

ybakos
Copy link
Contributor

@ybakos ybakos commented Apr 8, 2017

This series replaces 'later' with 'to the right of', mentions "indenting or positioning" for clarity, and corrects the name of the evaluate function in the "bad" example to match the function name in the "good" example.

@process-bot
Copy link

Thanks for the pull request! Make sure it satisfies this checklist. My human colleagues will appreciate it!

Here is what to expect next, and if anyone wants to comment, keep these things in mind.

@ybakos ybakos closed this Apr 8, 2017
@ybakos ybakos reopened this Apr 8, 2017
@ybakos
Copy link
Contributor Author

ybakos commented Apr 8, 2017

My local branch name should be 703-declarations-case, but I'll leave it as-is.
This branch addresses #703 .

Massage the writing for clarity.
Fixes elm#702.
…g into 702-declarations-case

Rebase from master, to ensure latest diffs / avoid merge conflicts
with upstream:master/HEAD.
@ybakos
Copy link
Contributor Author

ybakos commented Aug 23, 2019

@evancz I humbly request a quick review and merge of this documentation change to style-guide. Thanks for all things Elm!

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

2 participants