Skip to content

Restore CaseStringHelper as @api - #494

Merged
TomasVotruba merged 1 commit into
mainfrom
restore-case-string-helper
Jul 31, 2026
Merged

Restore CaseStringHelper as @api#494
TomasVotruba merged 1 commit into
mainfrom
restore-case-string-helper

Conversation

@TomasVotruba

Copy link
Copy Markdown
Member

CaseStringHelper was removed in #492 along with YamlToAttributeDoctrineMappingRector, but it is still used downstream by rector-drupal.

Restored as-is and marked @api, so class-leak skips it despite having no in-repo usage:

/**
 * @api used by rector-drupal
 */
final class CaseStringHelper
{
    public static function camelCase(string $value): string
    // ...
}

Covered with a unit test that pins current behavior:

yield ['some_property', 'someProperty'];
yield ['someProperty', 'someProperty'];
yield ['some property', 'someProperty'];
yield ['some-property', 'someproperty'];

@TomasVotruba
TomasVotruba enabled auto-merge (squash) July 31, 2026 20:51
@TomasVotruba
TomasVotruba disabled auto-merge July 31, 2026 20:52
@TomasVotruba
TomasVotruba merged commit a90a39f into main Jul 31, 2026
6 of 7 checks passed
@TomasVotruba
TomasVotruba deleted the restore-case-string-helper branch July 31, 2026 20:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant