Skip to content

chore(repo): update code owners - #2404

Draft
sriaradhyula wants to merge 1 commit into
mainfrom
prebuild/chore/update-codeowners
Draft

chore(repo): update code owners#2404
sriaradhyula wants to merge 1 commit into
mainfrom
prebuild/chore/update-codeowners

Conversation

@sriaradhyula

Copy link
Copy Markdown
Member

Description

Updates .github/CODEOWNERS to assign maintainers for dynamic agents, RAG, Helm charts, Slack bot, and Webex bot, with a shared default owner group for the rest of the repository.

The previous file contained stale Jira-specific text, only one default owner, and an outdated Slack bot owner list. This change aligns automatic review requests with the current component maintainers and canonical GitHub accounts.

Type of Change

  • Bugfix
  • New Feature
  • Breaking Change
  • Refactor
  • Documentation
  • Other: repository ownership configuration

Pre-release Helm Charts (Optional)

Not applicable. No chart content changed.

Checklist

  • I have read the contributing guidelines
  • Existing issues have been referenced (not applicable)
  • I have verified this change is not present in other open pull requests
  • Functionality is documented in the CODEOWNERS rules
  • All code style checks pass (git diff --check and CODEOWNERS syntax validation)
  • New code contribution is covered by automated tests (not applicable)
  • All new and existing tests pass (not applicable; configuration-only change)

Validation

  • Confirmed every component path exists.
  • Confirmed every listed owner has write-level access or higher.
  • Validated CODEOWNERS patterns and owner token syntax.

Signed-off-by: Sri Aradhyula <sraradhy@cisco.com>
@github-actions

Copy link
Copy Markdown
Contributor

⚠️ Proprietary Content Detected

This PR contains proprietary content that should not be in the open-source repository.

Found Proprietary Content:

\n\n📄 File: .github/CODEOWNERS\n\n2:* @sriaradhyula @subbaksh @kevkantes @cisco-erilutz @artdroz 5:/ai_platform_engineering/dynamic_agents/ @sriaradhyula @subbaksh @kevkantes @cisco-erilutz 6:/ai_platform_engineering/knowledge_bases/rag/ @sriaradhyula @cisco-erilutz @kevkantes @subbaksh 12:/ai_platform_engineering/integrations/slack_bot/ @cisco-erilutz @sriaradhyula @kevkantes\n

Allowed Patterns:

These patterns are allowed in specific files:

  • .cursorrules - Project-specific development rules
  • pyproject.toml - Maintainer information
  • .git_commit_msg.txt - Temporary commit message file
  • CONTRIBUTING.md - Contributor guidelines
  • CODE_OF_CONDUCT.md - Code of conduct

How to Fix:

  1. Generic Examples: Replace company-specific examples with generic ones:

    • cisco.com, outshift.io
    • example.com, your-domain.com
  2. Email Addresses: Use generic addresses:

    • user@cisco.com
    • user@example.com
  3. Group Names: Use generic group names:

    • eti-sre, cisco-eti
    • admin-group, your-admin-group
  4. Internal URLs: Remove or replace with placeholders:

    • https://outshift.io
    • https://your-internal-domain.com

Need Help?

If you believe these detections are false positives or should be allowed, please:

  1. Explain why in a comment on this PR
  2. Add the file to the exclusion list in .github/workflows/check-proprietary-content.yml

This PR cannot be merged until these violations are resolved.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Development

Successfully merging this pull request may close these issues.

1 participant