-
Notifications
You must be signed in to change notification settings - Fork 0
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: doc update #6
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #6 +/- ##
=======================================
Coverage 99.55% 99.55%
=======================================
Files 111 111
Lines 4244 4244
=======================================
Hits 4225 4225
Misses 19 19 ☔ View full report in Codecov by Sentry. |
f8f561d
to
18b0165
Compare
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.
Rather than adding many comments:
Some paths have a leading forward slash like:
/openstack/enums/...
Other paths do not have a leading forward slash:
openstack/enums/...
Which way is correct?
@khalford tere's no difference - its just a path - I'll remove the leading / since its more correct |
updating docs - remove mention of enums in user-docs to favor strings - makes it easier to use
remove leading / on paths
bab5de6
to
2181c84
Compare
updating docs - remove mention of enums in user-docs to favor strings
adding extra string aliases for presets