-
Notifications
You must be signed in to change notification settings - Fork 909
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
Update docs when generating new Datasource #5624
Conversation
2d65312
to
1311a87
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.
Looks good! Have one question but it is not blocking and more of a nit/potential suggestion so not gonna block on it
1. Add a new file in :file:`doc/rtd/reference/datasources/<cloudplatform>.rst` | ||
2. Reference `<cloudplatform>.rst` in :file:`doc/rtd/reference/datasources.rst` | ||
3. Add an alphebetized dsname entry in representing your datasource | ||
:file:`doc/rtd/reference/datasource_dsname_map.rst` |
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.
should there be a more direct call to action here to use the command found in doc/rtd/reference/datasource_dsname_map.rst
to generate the datasource list? Or is it just easier to tell the user to simply insert their new DS name in the list alphabetically (like you currently have)?
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.
It shouldn't need to be regenerated each time, so I think the instructions here make the most sense.
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!
1311a87
to
cfaf229
Compare
Minor updates to docs to tell DS creators they need to add dsname entries to datasource_dsname_map.rst
Proposed Commit Message
See individual commits
Additional Context
Test Steps
Merge type