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

Improve headings in arrays section of the manual #27769

Merged
merged 1 commit into from
Jun 27, 2018
Merged

Conversation

nalimilan
Copy link
Member

Having a single section isn't useful: remove redundant "Arrays" heading and move all sub-headings one level higher.

@nalimilan nalimilan added docs This change adds or pertains to documentation arrays [a, r, r, a, y, s] labels Jun 25, 2018
@fredrikekre
Copy link
Member

I think there might be references to the ## Arrays header that you are removing here, did the docs build for you locally with no errors?

Having a single section isn't useful: remove redundant "Arrays" heading and move all sub-headings one level higher.

[ci skip]
@nalimilan
Copy link
Member Author

Good catch, a reference was missing. Should be fixed now.

@nalimilan nalimilan merged commit dfe3443 into master Jun 27, 2018
@nalimilan nalimilan deleted the nl/arraydocs branch June 27, 2018 08:40
jrevels pushed a commit that referenced this pull request Jul 2, 2018
Having a single section isn't useful: remove redundant "Arrays" heading and move all sub-headings one level higher.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
arrays [a, r, r, a, y, s] docs This change adds or pertains to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants