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

Tiny documentation fix. #5659

Merged
merged 1 commit into from Apr 26, 2017
Merged

Tiny documentation fix. #5659

merged 1 commit into from Apr 26, 2017

Conversation

BluebirdGreycoat
Copy link
Contributor

Splitting the function declaration across two lines, and marking the second line with a star, seems inconsistent and can be momentarily confusing.

There are plenty of lines longer than 80 characters, and spliting the function declaration across two lines can be momentarily confusing.
@SmallJoker SmallJoker added @ Script API Trivial The change is a trivial bug fix, documentation or maintenance change, as per the Git Guidelines labels Apr 25, 2017
@paramat
Copy link
Contributor

paramat commented Apr 25, 2017

👍

@nerzhul nerzhul merged commit 70ceeb8 into minetest:master Apr 26, 2017
@BluebirdGreycoat BluebirdGreycoat deleted the doc_fix branch April 26, 2017 15:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
@ Documentation @ Script API Trivial The change is a trivial bug fix, documentation or maintenance change, as per the Git Guidelines >= Two approvals ✅ ✅
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants