-
-
Notifications
You must be signed in to change notification settings - Fork 210
doc: align module capitalization #1115
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
doc: align module capitalization #1115
Conversation
82d91eb
to
b174420
Compare
13ae07d
to
c45c759
Compare
f04195e
to
097a68a
Compare
as an aside, we should consider breaking |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Overall approach LGTM.
Co-authored-by: NAHO <90870942+trueNAHO@users.noreply.github.com>
eec3ff4
to
4a51deb
Compare
The way the index works means the links in the sidebar are sorted into alphabetical order based on the module slug: it would be nice to change that to use the human readable name at some point. Not necessary in this PR though. |
Co-authored-by: Daniel Thwaites <danth@danth.me>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
Add required meta.name attribute as of commit 82f67a3 ("doc: align module capitalization (danth#1115)").
changes:
meta.name
attributemeta.name
attribute to all applicable meta filesmeta.name
attributerelated: #1084
Things done
Notify maintainers