Skip to content

Conversation

Kocal
Copy link
Member

@Kocal Kocal commented Oct 28, 2024

Q A
Bug fix? no
New feature? no
Issues Fix #...
License MIT

As discussed internally, I'm adding myself as a code owner for some UX packages.
I'm not a fan of the "code owner" term, but the goal here is to be automatically requested when a pull-request on those packages is opened.

Feel free to open a new PR and add yourself as code-owners if you want :)

@carsonbot carsonbot added the Status: Needs Review Needs to be reviewed label Oct 28, 2024
@Kocal Kocal merged commit d100df4 into symfony:2.x Oct 28, 2024
58 of 59 checks passed
@Kocal Kocal deleted the codeowners branch October 28, 2024 20:24
Kocal added a commit that referenced this pull request May 31, 2025
For the same reasons explained in #2308
Kocal added a commit that referenced this pull request May 31, 2025
This PR was merged into the 2.x branch.

Discussion
----------

Add myself to Toolkit code-owners

| Q             | A
| ------------- | ---
| Bug fix?      | no
| New feature?  | no <!-- please update src/**/CHANGELOG.md files -->
| Docs?         | no <!-- required for new features -->
| Issues        | Fix #... <!-- prefix each issue number with "Fix #", no need to create an issue if none exist, explain below instead -->
| License       | MIT

<!--
Replace this notice by a description of your feature/bugfix.
This will help reviewers and should be a good start for the documentation.

Additionally (see https://symfony.com/releases):
 - Always add tests and ensure they pass.
 - For new features, provide some code snippets to help understand usage.
 - Features and deprecations must be submitted against branch main.
 - Update/add documentation as required (we can help!)
 - Changelog entry should follow https://symfony.com/doc/current/contributing/code/conventions.html#writing-a-changelog-entry
 - Never break backward compatibility (see https://symfony.com/bc).
-->

For the same reasons explained in #2308

Commits
-------

39c550d Add myself to Toolkit code-owners
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Status: Needs Review Needs to be reviewed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants