Skip to content

Decorate stdClass type#24

Merged
ruudk merged 3 commits into
TicketSwap:mainfrom
simbig:add-stdClass-as-type
Sep 24, 2025
Merged

Decorate stdClass type#24
ruudk merged 3 commits into
TicketSwap:mainfrom
simbig:add-stdClass-as-type

Conversation

@simbig

@simbig simbig commented Sep 24, 2025

Copy link
Copy Markdown
Contributor

Add missing stdClass to the type-regex.

Wird current master I see the follwing:
image

@ghost ghost added the work in progress label Sep 24, 2025
@simbig
simbig marked this pull request as ready for review September 24, 2025 09:04
@ruudk

ruudk commented Sep 24, 2025

Copy link
Copy Markdown
Member

Thanks @simbig, could you add a simple test case here?

yield [
"Array has 3 duplicate keys with value '<fg=yellow>App\Activity</>' (<fg=yellow>\App\Activity</>::class).",
"Array has 3 duplicate keys with value 'App\Activity' (\App\Activity::class).",
null,
null,
true
];

@ruudk ruudk changed the title add stdClass as type Decorate stdClass type Sep 24, 2025
@ruudk

ruudk commented Sep 24, 2025

Copy link
Copy Markdown
Member

@simbig Small issue with the assertion.

@ruudk
ruudk merged commit 32514df into TicketSwap:main Sep 24, 2025
16 checks passed
@simbig

simbig commented Sep 24, 2025

Copy link
Copy Markdown
Contributor Author

@ruudk fixed it. thanks for the quick response

@simbig
simbig deleted the add-stdClass-as-type branch September 24, 2025 09:20
@ruudk

ruudk commented Sep 24, 2025

Copy link
Copy Markdown
Member

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.

2 participants