Skip to content

ci: Add codeowner path to ci-docs.yaml file#502

Merged
vsiravar merged 2 commits into
runfinch:mainfrom
vsiravar:vsiravar/add-codeowners-ci-docs
Jul 25, 2023
Merged

ci: Add codeowner path to ci-docs.yaml file#502
vsiravar merged 2 commits into
runfinch:mainfrom
vsiravar:vsiravar/add-codeowners-ci-docs

Conversation

@vsiravar
Copy link
Copy Markdown
Contributor

Issue #, if available:
Add codeowners file path to ci-docs.yaml to prevent CI from getting blocked.
Description of changes:
Fixes:
Testing done:

N/A

  • I've reviewed the guidance in CONTRIBUTING.md

License Acceptance

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Signed-off-by: Vishwas Siravara <siravara@amazon.com>
@vsiravar vsiravar requested a review from weikequ July 25, 2023 01:26
@vsiravar vsiravar requested a review from a team as a code owner July 25, 2023 01:26
@vsiravar vsiravar mentioned this pull request Jul 25, 2023
1 task
Comment thread .github/workflows/ci-docs.yaml Outdated
Signed-off-by: Vishwas Siravara <siravara@amazon.com>
Copy link
Copy Markdown
Contributor

@austinvazquez austinvazquez left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM thanks!

Copy link
Copy Markdown
Contributor

@weikequ weikequ left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Thanks!

@vsiravar
Copy link
Copy Markdown
Contributor Author

Flaky tests, no impact on this change since there is no code change or workflow change.

@vsiravar vsiravar merged commit 81d8b20 into runfinch:main Jul 25, 2023
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.

3 participants