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

CODEOWNERS: Add sig-foundations #24976

Merged
merged 1 commit into from May 24, 2023

Conversation

joamaki
Copy link
Contributor

@joamaki joamaki commented Apr 19, 2023

Assign pkg/{hive,statedb,promise,stream} and daemon/cmd/cells.go to sig-foundations.

@joamaki joamaki requested a review from a team as a code owner April 19, 2023 10:40
@joamaki joamaki requested a review from qmonnet April 19, 2023 10:40
@maintainer-s-little-helper maintainer-s-little-helper bot added the dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. label Apr 19, 2023
@joamaki joamaki requested a review from pchaigno April 19, 2023 10:40
@joamaki joamaki added the release-note/ci This PR makes changes to the CI. label Apr 19, 2023
@maintainer-s-little-helper maintainer-s-little-helper bot removed the dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. label Apr 19, 2023
CODEOWNERS Outdated Show resolved Hide resolved
Copy link
Member

@pchaigno pchaigno left a comment

Choose a reason for hiding this comment

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

Some of the entries you reordered were in non-alphabetical order because GitHub applies the ownership in the order they are defined. See comment below.

EDIT: Maybe also best to keep those changes in a separate commit.

CODEOWNERS Show resolved Hide resolved
CODEOWNERS Outdated Show resolved Hide resolved
CODEOWNERS Outdated Show resolved Hide resolved
CODEOWNERS Outdated Show resolved Hide resolved
CODEOWNERS Show resolved Hide resolved
CODEOWNERS Show resolved Hide resolved
@joamaki joamaki force-pushed the pr/joamaki/foundations-codeowners branch from 0291084 to 347db46 Compare April 20, 2023 06:22
@joamaki joamaki requested a review from a team as a code owner April 20, 2023 06:22
Copy link
Contributor

@zacharysarah zacharysarah left a comment

Choose a reason for hiding this comment

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

+1 to consideration for @joestringer's comment, otherwise LGTM from a docs perspective.

@qmonnet qmonnet requested review from qmonnet and removed request for qmonnet April 25, 2023 08:42
@joamaki joamaki self-assigned this Apr 26, 2023
@qmonnet
Copy link
Member

qmonnet commented May 19, 2023

Given that there is feedback to address and a conflict to solve, I'm turning this PR to draft status for the time being.

Assign pkg/{hive,statedb,promise,stream} and daemon/cmd/cells.go to
sig-foundations.

Signed-off-by: Jussi Maki <jussi@isovalent.com>
@joamaki joamaki force-pushed the pr/joamaki/foundations-codeowners branch from 347db46 to a001fbd Compare May 24, 2023 14:01
@joamaki joamaki marked this pull request as ready for review May 24, 2023 14:35
@joamaki
Copy link
Contributor Author

joamaki commented May 24, 2023

Some of the entries you reordered were in non-alphabetical order because GitHub applies the ownership in the order they are defined. See comment below.

EDIT: Maybe also best to keep those changes in a separate commit.

Reverted all the reorderings.

Copy link
Member

@pchaigno pchaigno left a comment

Choose a reason for hiding this comment

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

One comment above is still relevant (GitHub warning). Otherwise, LGTM.

@joestringer
Copy link
Member

The GitHub warning should be resolved now.

@joestringer joestringer merged commit 4febdd9 into cilium:main May 24, 2023
45 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release-note/ci This PR makes changes to the CI.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants