Skip to content

[air] update /python/ray/air CODEOWNERS#55891

Merged
justinvyu merged 1 commit intoray-project:masterfrom
matthewdeng:air-owners
Aug 28, 2025
Merged

[air] update /python/ray/air CODEOWNERS#55891
justinvyu merged 1 commit intoray-project:masterfrom
matthewdeng:air-owners

Conversation

@matthewdeng
Copy link
Copy Markdown
Contributor

Add Ray Train maintainers as the CODEOWNERS for /python/ray/air sub-directory.

Without this change, this directory will have Ray Core assigned as the CODEOWNER due to this line.

Signed-off-by: Matthew Deng <matt@anyscale.com>
@matthewdeng matthewdeng requested a review from justinvyu August 25, 2025 17:19
Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-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.

Code Review

This pull request updates the .github/CODEOWNERS file to assign ownership of the /python/ray/air/ directory to the Ray Train team. The changes also include clarifying comments for the Ray Tune and Ray Train sections, which improves readability. The main change is correct and aligns with the PR's goal. I've added one suggestion to also include the documentation path for ray.air to maintain consistency with other components in the file.

/doc/source/train/ @ray-project/ray-train @ray-project/ray-docs

# Ray AIR
/python/ray/air/ @ray-project/ray-train
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

medium

For consistency with other components like Tune and Train, consider also adding an entry for the ray.air documentation. This would ensure that the Ray Train team is also the owner for any future documentation related to ray.air.

/python/ray/air/ @ray-project/ray-train
/doc/source/air/ @ray-project/ray-train @ray-project/ray-docs

@ray-gardener ray-gardener bot added docs An issue or change related to documentation train Ray Train Related Issue labels Aug 25, 2025
@matthewdeng matthewdeng added the go add ONLY when ready to merge, run all tests label Aug 25, 2025
@justinvyu justinvyu merged commit d3597cb into ray-project:master Aug 28, 2025
5 checks passed
tohtana pushed a commit to tohtana/ray that referenced this pull request Aug 29, 2025
Add Ray Train maintainers as the CODEOWNERS for `/python/ray/air`
sub-directory.

Without this change, this directory will have Ray Core assigned as the
CODEOWNER due to [this
line](https://github.com/matthewdeng/ray/blob/ca2d866a95385aea4fd3de1c3b9f5148f899c282/.github/CODEOWNERS#L20-L21).

Signed-off-by: Matthew Deng <matt@anyscale.com>
Signed-off-by: Masahiro Tanaka <mtanaka@anyscale.com>
tohtana pushed a commit to tohtana/ray that referenced this pull request Aug 29, 2025
Add Ray Train maintainers as the CODEOWNERS for `/python/ray/air`
sub-directory.

Without this change, this directory will have Ray Core assigned as the
CODEOWNER due to [this
line](https://github.com/matthewdeng/ray/blob/ca2d866a95385aea4fd3de1c3b9f5148f899c282/.github/CODEOWNERS#L20-L21).

Signed-off-by: Matthew Deng <matt@anyscale.com>
Signed-off-by: Masahiro Tanaka <mtanaka@anyscale.com>
gangsf pushed a commit to gangsf/ray that referenced this pull request Sep 2, 2025
Add Ray Train maintainers as the CODEOWNERS for `/python/ray/air`
sub-directory.

Without this change, this directory will have Ray Core assigned as the
CODEOWNER due to [this
line](https://github.com/matthewdeng/ray/blob/ca2d866a95385aea4fd3de1c3b9f5148f899c282/.github/CODEOWNERS#L20-L21).

Signed-off-by: Matthew Deng <matt@anyscale.com>
Signed-off-by: Gang Zhao <gang@gang-JQ62HD2C37.local>
sampan-s-nayak pushed a commit to sampan-s-nayak/ray that referenced this pull request Sep 8, 2025
Add Ray Train maintainers as the CODEOWNERS for `/python/ray/air`
sub-directory.

Without this change, this directory will have Ray Core assigned as the
CODEOWNER due to [this
line](https://github.com/matthewdeng/ray/blob/ca2d866a95385aea4fd3de1c3b9f5148f899c282/.github/CODEOWNERS#L20-L21).

Signed-off-by: Matthew Deng <matt@anyscale.com>
Signed-off-by: sampan <sampan@anyscale.com>
jugalshah291 pushed a commit to jugalshah291/ray_fork that referenced this pull request Sep 11, 2025
Add Ray Train maintainers as the CODEOWNERS for `/python/ray/air`
sub-directory.

Without this change, this directory will have Ray Core assigned as the
CODEOWNER due to [this
line](https://github.com/matthewdeng/ray/blob/ca2d866a95385aea4fd3de1c3b9f5148f899c282/.github/CODEOWNERS#L20-L21).

Signed-off-by: Matthew Deng <matt@anyscale.com>
Signed-off-by: jugalshah291 <shah.jugal291@gmail.com>
wyhong3103 pushed a commit to wyhong3103/ray that referenced this pull request Sep 12, 2025
Add Ray Train maintainers as the CODEOWNERS for `/python/ray/air`
sub-directory.

Without this change, this directory will have Ray Core assigned as the
CODEOWNER due to [this
line](https://github.com/matthewdeng/ray/blob/ca2d866a95385aea4fd3de1c3b9f5148f899c282/.github/CODEOWNERS#L20-L21).

Signed-off-by: Matthew Deng <matt@anyscale.com>
Signed-off-by: yenhong.wong <yenhong.wong@grabtaxi.com>
dstrodtman pushed a commit that referenced this pull request Oct 6, 2025
Add Ray Train maintainers as the CODEOWNERS for `/python/ray/air`
sub-directory.

Without this change, this directory will have Ray Core assigned as the
CODEOWNER due to [this
line](https://github.com/matthewdeng/ray/blob/ca2d866a95385aea4fd3de1c3b9f5148f899c282/.github/CODEOWNERS#L20-L21).

Signed-off-by: Matthew Deng <matt@anyscale.com>
Signed-off-by: Douglas Strodtman <douglas@anyscale.com>
landscapepainter pushed a commit to landscapepainter/ray that referenced this pull request Nov 17, 2025
Add Ray Train maintainers as the CODEOWNERS for `/python/ray/air`
sub-directory.

Without this change, this directory will have Ray Core assigned as the
CODEOWNER due to [this
line](https://github.com/matthewdeng/ray/blob/ca2d866a95385aea4fd3de1c3b9f5148f899c282/.github/CODEOWNERS#L20-L21).

Signed-off-by: Matthew Deng <matt@anyscale.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs An issue or change related to documentation go add ONLY when ready to merge, run all tests train Ray Train Related Issue

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants