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

Add cms-platform to CODEOWNERS file #396

Merged
merged 1 commit into from
Apr 6, 2023
Merged

Conversation

jazzsequence
Copy link
Contributor

No description provided.

@jazzsequence jazzsequence requested a review from a team April 5, 2023 20:45
@jazzsequence jazzsequence self-assigned this Apr 5, 2023
@jazzsequence jazzsequence requested a review from a team as a code owner April 5, 2023 20:45
@CdrMarks
Copy link

CdrMarks commented Apr 6, 2023

I'm -1 on this change as it creates duplicate notifications for those of us in both teams.
image

@jazzsequence
Copy link
Contributor Author

@CdrMarks As CMS Platform is the team that maintains this, it feels odd that we don't "own" this plugin (and all others). GitHub streaming notifications into Slack are already a firehose and I don't think it's worth restricting our ownership to better benefit that specific integration. Whether we want to remove CMS Operations in favor of using CMS Platform as the owner is a different question.

@jazzsequence jazzsequence merged commit 4b608de into develop Apr 6, 2023
@jazzsequence jazzsequence deleted the update-codeowners branch April 6, 2023 17:09
jazzsequence added a commit that referenced this pull request Apr 6, 2023
* [CMS-1033] Add GH action for deployments to wp.org (#368)

* Make dependabot target develop branch (#376)

* Declare wp_cache_supports function and support features (#378)

* Fix broken wp_cache_supports() (#382)

* Update images for lint and test-behat jobs

* Bump yoast/phpunit-polyfills from 1.0.3 to 1.0.4

* Bump dealerdirect/phpcodesniffer-composer-installer from 0.7.2 to 1.0.0

Bumps [dealerdirect/phpcodesniffer-composer-installer](https://github.com/Dealerdirect/phpcodesniffer-composer-installer) from 0.7.2 to 1.0.0.
- [Release notes](https://github.com/Dealerdirect/phpcodesniffer-composer-installer/releases)
- [Changelog](https://github.com/PHPCSStandards/composer-installer/blob/main/.github_changelog_generator)
- [Commits](PHPCSStandards/composer-installer@v0.7.2...v1.0.0)

---
updated-dependencies:
- dependency-name: dealerdirect/phpcodesniffer-composer-installer
  dependency-type: direct:development
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <[email protected]>

* Bump grunt from 1.5.2 to 1.6.1 (#386)

* [CMSP-223] WP Redis housekeeping (#387)

* [CMS-1033] Release 1.3.0 (#371)

* [BUGS-5583] Release 1.3.1 (#379)

* Release 1.3.2 (#383)

* add spdy-compatible license version

* remove reference to package.json

---------

Co-authored-by: John Spellman <[email protected]>

* [CMSP-72] PHP 8.2 compatibility (#388)

* [BUGS-5623] Set missing_redis_message if Redis service is not connected (#391)

* Set missing_redis_message if Redis is not connected

* i18n

* Bump yoast/phpunit-polyfills from 1.0.4 to 1.0.5 (#394)

Bumps [yoast/phpunit-polyfills](https://github.com/Yoast/PHPUnit-Polyfills) from 1.0.4 to 1.0.5.
- [Release notes](https://github.com/Yoast/PHPUnit-Polyfills/releases)
- [Changelog](https://github.com/Yoast/PHPUnit-Polyfills/blob/2.x/CHANGELOG.md)
- [Commits](Yoast/PHPUnit-Polyfills@1.0.4...1.0.5)

---
updated-dependencies:
- dependency-name: yoast/phpunit-polyfills
  dependency-type: direct:development
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* version bump

* update changelogs

* add chris and john to contributors

* Add cms-platform to CODEOWNERS file (#396)

* update date

---------

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: John Spellman <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
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