Skip to content

feat: add code owners documentation #546

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

Merged
merged 6 commits into from
May 2, 2024
Merged

feat: add code owners documentation #546

merged 6 commits into from
May 2, 2024

Conversation

xmfcx
Copy link
Contributor

@xmfcx xmfcx commented May 2, 2024

Description

Pre-review checklist for the PR author

The PR author must check the checkboxes below when creating the PR.

In-review checklist for the PR reviewers

The Reviewers must check the checkboxes below before approval.

Post-review checklist for the PR author

The PR author must check the checkboxes below before merging.

  • There are no open discussions or they are tracked via tickets.

After all checkboxes are checked, anyone who has write access can merge the PR.

Signed-off-by: M. Fatih Cırıt <mfc@leodrive.ai>
@xmfcx xmfcx added type:documentation Creating or refining documentation. tag:deploy-docs Mark for deploy-docs action generation. (used-by-ci) labels May 2, 2024
@xmfcx xmfcx self-assigned this May 2, 2024
Copy link
Contributor

@esteve esteve left a comment

Choose a reason for hiding this comment

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

@xmfcx LGTM, thanks

Signed-off-by: M. Fatih Cırıt <mfc@leodrive.ai>
@xmfcx xmfcx marked this pull request as draft May 2, 2024 11:27
@esteve
Copy link
Contributor

esteve commented May 2, 2024

@xmfcx I've reviewed this PR before the latest changes 😅 Are there more changes coming or should I review the PR as is now?

@xmfcx
Copy link
Contributor Author

xmfcx commented May 2, 2024

So fast @esteve :D I am also reviewing it with chatgpt and will do some wording and formatting improvements with that but the core information will stay the same.

M. Fatih Cırıt added 2 commits May 2, 2024 14:40
Signed-off-by: M. Fatih Cırıt <mfc@leodrive.ai>
Signed-off-by: M. Fatih Cırıt <mfc@leodrive.ai>
@xmfcx xmfcx marked this pull request as ready for review May 2, 2024 11:43
@xmfcx
Copy link
Contributor Author

xmfcx commented May 2, 2024

Alright it's ready now 😁

@xmfcx
Copy link
Contributor Author

xmfcx commented May 2, 2024

Signed-off-by: M. Fatih Cırıt <mfc@leodrive.ai>
@xmfcx xmfcx closed this May 2, 2024
@xmfcx xmfcx reopened this May 2, 2024
@xmfcx
Copy link
Contributor Author

xmfcx commented May 2, 2024

I'll merge it for now, if you have any reviews or comments, I will open up a new PR to also apply them.

@xmfcx xmfcx merged commit 86aa0c3 into main May 2, 2024
10 of 12 checks passed
@xmfcx xmfcx deleted the feat/code-owners-docs branch May 2, 2024 12:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
tag:deploy-docs Mark for deploy-docs action generation. (used-by-ci) type:documentation Creating or refining documentation.
Projects
No open projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants