-
Notifications
You must be signed in to change notification settings - Fork 32
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 readme with required sections #686
Conversation
Signed-off-by: Bryan Behrenshausen <[email protected]>
7472177
to
5742c6f
Compare
Signed-off-by: Bryan Behrenshausen <[email protected]>
5742c6f
to
432787f
Compare
Nothing urgent here, @gsmith-sas, but since this PR does actually add a few sections and specifications to the README, I wanted to give you the opportunity to have a look. |
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 move the links to the project's documentation to a separate Additional resources section, I'd like to keep it with the project info (now under the Overview section). We moved virtually all of the project's documentation to the Help Center and want to make sure users know where to find it.
So, I'm asking that we move the new Contributing and License sections to after the proposed Additional resources section an remove the Additional resources section header.
Signed-off-by: Bryan Behrenshausen <[email protected]>
Sure thing, @gsmith-sas. Makes sense to 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.
Thanks for the changes. LGTM.
This pull request adds a few new sections to the project README. It does so primarily to test automations associated with the DCO bot.