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

change(control): Typofixes and new tests for RDMS #617

Merged
merged 6 commits into from
Jan 20, 2025

Conversation

shoffman-percona
Copy link
Contributor

Removed outdated markdown file
Corrected typos, Added additional tests
Also squashed changes to single commit to address incorrect name/email for CLA check.

…te. Removed outdated markdown file

Corrected typos, Added additional tests
@shoffman-percona shoffman-percona requested review from a team as code owners January 10, 2025 22:59
@shoffman-percona
Copy link
Contributor Author

Hallelujah! finally a PR that passes the EasyCLA check on all (1) commits!

@eddie-knight
Copy link
Contributor

Thanks for this @shoffman-percona!

@robmoffat — I'm surprised to see netlify running on this PR. Could you look into this?

@mlysaght2017
Copy link
Contributor

thanks @shoffman-percona! Can you please ensure the lint-checker passes also.

@eddie-knight
Copy link
Contributor

@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.

@eddie-knight eddie-knight changed the title Cleanup, typos and extending features change(control): Typofixes and new tests for RDMS Jan 14, 2025
@finos finos deleted a comment from netlify bot Jan 14, 2025
@eddie-knight
Copy link
Contributor

@mlysaght2017 do we want to adjust the CODEOWNERS file so that it doesn't require a review from Taxonomy WG?

@eddie-knight eddie-knight merged commit 191f3f6 into finos:main Jan 20, 2025
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants