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

Add CODEOWNERS #154

Merged
merged 7 commits into from
Dec 19, 2024
Merged

Conversation

RajeshThallam
Copy link
Collaborator

@RajeshThallam RajeshThallam commented Dec 19, 2024

Description

Add CODEOWNERS file

Fixes # (if applicable)

Checklist

  • Contribution Guidelines: I have read the Contribution Guidelines.
  • CLA: I have signed the CLA.
  • Authorship: I am listed as the author (if applicable).
  • Conventional Commits: My PR title and commit messages follow the Conventional Commits spec.
  • Code Format: I have run nox -s format to format the code.
  • Spelling: I have fixed any spelling errors, and added false positives to .github/actions/spelling/allow.txt if necessary.
  • Template: I have followed the aaie_notebook_template.ipynb if submitting a new jupyter notbook.
  • Sync: My Fork is synced with the upstream.
  • Documentations: I have updated relevant documentations (if applicable) in the docs folder.

Copy link

@code-review-assist code-review-assist bot left a comment

Choose a reason for hiding this comment

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

Hi RajeshThallam and reviewers,

I'm currently reviewing this pull request and will post my detailed review in a few minutes. In the meantime, here's a quick summary to help everyone get up to speed:

This pull request adds a CODEOWNERS file to the applied-ai-engineering-samples repository. The intent is to define code ownership for better collaboration and maintainability. Specifically:

  • A CODEOWNERS file has been created at .github/CODEOWNERS.
  • This file designates @RajeshThallam and @Abhishekbhagwat as the default code owners for the entire repository.

The update_notebook_links.py file has been removed. This appears to be an unrelated change and may warrant a separate pull request.

I'll provide a more thorough review shortly, but this should give you a good overview of the changes.

And now, a little haiku to lighten the mood:

Code flows like a stream,
Owners guide its gentle path,
Clarity takes hold.

Thanks,
Gemini

@Abhishekbhagwat Abhishekbhagwat merged commit 2225f1c into GoogleCloudPlatform:main Dec 19, 2024
3 of 4 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.

2 participants