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

docs: remove redundant "the" from SDS readme #9302

Closed
wants to merge 1 commit into from

Conversation

stew-r
Copy link

@stew-r stew-r commented Dec 10, 2019

Description: a minor tweak to the SDS readme which removes a redundant word
Risk Level: low (docs)
Testing: none
Docs Changes: N/A
Release Notes: not user or extension developer impacting

@snowp
Copy link
Contributor

snowp commented Dec 10, 2019

@stew-r
Copy link
Author

stew-r commented Dec 10, 2019

Can I do this from the GitHub UI?

@dio
Copy link
Member

dio commented Dec 10, 2019

@stew-r We're not aware of it. And, re: the fix, can you find similar issues to this?

@stew-r
Copy link
Author

stew-r commented Dec 10, 2019

Sorry, I thought this would be an easy/quick fix. I can revert if there isn't a straightforward way to incorporate this small docs change.

@stale
Copy link

stale bot commented Dec 23, 2019

This pull request has been automatically marked as stale because it has not had activity in the last 7 days. It will be closed in 7 days if no further activity occurs. Please feel free to give a status update now, ping for review, or re-open when it's ready. Thank you for your contributions!

@stale stale bot added the stale stalebot believes this issue/PR has not been touched recently label Dec 23, 2019
@stew-r
Copy link
Author

stew-r commented Dec 23, 2019

I'm still interested in making this (and future) improvements to docs. It doesn't seem like it should be hard to make a minor text change. Please let me know if there's a straightforward way to get that done.

@stale stale bot removed the stale stalebot believes this issue/PR has not been touched recently label Dec 23, 2019
@dio
Copy link
Member

dio commented Dec 23, 2019

@stew-r Sorry that DCO is required. A related issue, re: from GitHub UI: todogroup/gh-issues#50.

@stew-r
Copy link
Author

stew-r commented Dec 23, 2019

Got it. Thanks for following up -- that's a shame. Closing this PR.

@stew-r stew-r closed this Dec 23, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants