Skip to content

Conversation

@shubham-n-khanna
Copy link
Contributor

Description

Added code owner

Types of changes

  • Docs change / refactoring / dependency upgrade
  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist

  • My code follows the code style of this project.
  • My change requires a change to the documentation and I have updated the documentation accordingly.
  • I have added tests to cover my changes.
  • I have checked downstream dependencies (e.g. ExternalTaskSensors) by searching for DAG name elsewhere in the repo

Copilot AI review requested due to automatic review settings December 8, 2025 12:34
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR adds a CODEOWNERS file to establish code ownership for the repository, assigning all files to the @patterninc/data-acquisition team by default.

  • Introduces a .github/CODEOWNERS file with global ownership pattern
  • Sets @patterninc/data-acquisition team as the default owners for all files
  • Includes descriptive comments explaining the ownership structure

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@@ -0,0 +1,4 @@
# code owners for the directories
Copy link

Copilot AI Dec 8, 2025

Choose a reason for hiding this comment

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

[nitpick] The comment "code owners" should use proper capitalization to match the standard terminology. Consider changing to "Code owners" or "CODEOWNERS" for better professional appearance and consistency with GitHub's terminology.

Suggested change
# code owners for the directories
# Code owners for the directories

Copilot uses AI. Check for mistakes.
@shubham-n-khanna shubham-n-khanna merged commit 2531865 into main Dec 8, 2025
12 checks passed
@shubham-n-khanna shubham-n-khanna deleted the shubham-n-khanna-patch-1 branch December 8, 2025 12:36
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