-
Notifications
You must be signed in to change notification settings - Fork 43
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
change(control): Typofixes and new tests for RDMS #617
Conversation
…te. Removed outdated markdown file Corrected typos, Added additional tests
Hallelujah! finally a PR that passes the EasyCLA check on all (1) commits! |
Thanks for this @shoffman-percona! @robmoffat — I'm surprised to see netlify running on this PR. Could you look into this? |
thanks @shoffman-percona! Can you please ensure the lint-checker passes also. |
@mlysaght2017 — The link checker was reporting a failure from the OSCAL training resource, on the gitter url. That link seemed functional when I looked, so I just hit the button to re-run the job and it's passing fine now. @damienjburks / @finos/ccc-wg-delivery — We'll need to circle back to the link checker to ensure it only evaluates changed files in the future. |
@mlysaght2017 do we want to adjust the CODEOWNERS file so that it doesn't require a review from Taxonomy WG? |
Removed outdated markdown file
Corrected typos, Added additional tests
Also squashed changes to single commit to address incorrect name/email for CLA check.