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

Drop deprecated OWNER from ya.make files #4742

Merged
merged 1 commit into from
May 27, 2024

Conversation

thevery
Copy link
Contributor

@thevery thevery commented May 22, 2024

Changelog entry

Drop deprecated OWNER from ya.make files

Changelog category

  • Not for changelog (changelog entry is not required)

@thevery thevery requested review from a team as code owners May 22, 2024 08:59
Copy link

Hi! Thank you for contributing!
The tests on this PR will run after a maintainer adds an ok-to-test label to this PR manually. Thank you for your patience!

@maximyurchuk maximyurchuk added the ok-to-test Special label used to approve a PR for testing on our infrastructure label May 22, 2024
@github-actions github-actions bot removed the ok-to-test Special label used to approve a PR for testing on our infrastructure label May 22, 2024
@thevery thevery force-pushed the owners-to-subscribers branch from 39f5637 to eff2706 Compare May 22, 2024 09:12
@thevery thevery changed the title Convert OWNER to SUBSCRIBER in ya.make Drop deprecated OWNER from ya.make files May 22, 2024
@thevery thevery force-pushed the owners-to-subscribers branch 4 times, most recently from b20019e to 0de1b49 Compare May 22, 2024 09:21
@maximyurchuk maximyurchuk added the ok-to-test Special label used to approve a PR for testing on our infrastructure label May 22, 2024
@github-actions github-actions bot removed the ok-to-test Special label used to approve a PR for testing on our infrastructure label May 22, 2024
Copy link

github-actions bot commented May 22, 2024

2024-05-22 09:35:41 UTC Pre-commit check for 56bba90 has started.
2024-05-22 09:35:44 UTC Build linux-x86_64-release-clang14 is running...
🟢 2024-05-22 09:36:52 UTC Build successful.

Copy link

github-actions bot commented May 22, 2024

2024-05-22 09:36:01 UTC Pre-commit check for 56bba90 has started.
2024-05-22 09:36:03 UTC Build linux-x86_64-release-asan is running...
🟢 2024-05-22 09:37:18 UTC Build successful.
2024-05-22 09:38:51 UTC Tests are running...
🔴 2024-05-22 09:49:56 UTC Some tests failed, follow the links below.

Test history

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
644 633 0 2 9 0

Copy link

github-actions bot commented May 22, 2024

2024-05-22 09:36:03 UTC Pre-commit check for 56bba90 has started.
2024-05-22 09:36:07 UTC Build linux-x86_64-relwithdebinfo is running...
🟢 2024-05-22 09:37:23 UTC Build successful.
2024-05-22 09:38:51 UTC Tests are running...
🟢 2024-05-22 09:45:51 UTC Tests successful.

Test history

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
794 785 0 0 9 0

uzhastik
uzhastik previously approved these changes May 23, 2024
@thevery thevery requested a review from a team as a code owner May 24, 2024 08:56
@thevery thevery requested a review from niksaveliev May 24, 2024 08:56
@thevery thevery force-pushed the owners-to-subscribers branch from e8f8d4f to 657fef6 Compare May 24, 2024 09:05
@thevery thevery requested a review from vitstn May 24, 2024 14:52
@maximyurchuk maximyurchuk removed the request for review from niksaveliev May 27, 2024 10:34
@maximyurchuk maximyurchuk added the rebase-and-check Rebase PR with the current base branch and check label May 27, 2024
@github-actions github-actions bot removed the rebase-and-check Rebase PR with the current base branch and check label May 27, 2024
@maximyurchuk maximyurchuk merged commit bc3818b into ydb-platform:main May 27, 2024
7 of 8 checks passed
@thevery thevery deleted the owners-to-subscribers branch May 27, 2024 10:56
@niksaveliev niksaveliev mentioned this pull request May 29, 2024
@StekPerepolnen StekPerepolnen mentioned this pull request May 31, 2024
@niksaveliev niksaveliev mentioned this pull request Jun 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants