Skip to content

Comments

[php 8.0] Check if class is an attribute in AnnotationToAttributeRector, to avoid non-attribute bug#6954

Merged
TomasVotruba merged 1 commit intomainfrom
tv-skip-non-attribute
Jun 2, 2025
Merged

[php 8.0] Check if class is an attribute in AnnotationToAttributeRector, to avoid non-attribute bug#6954
TomasVotruba merged 1 commit intomainfrom
tv-skip-non-attribute

Conversation

@TomasVotruba
Copy link
Member

@TomasVotruba TomasVotruba commented Jun 2, 2025

No description provided.

@TomasVotruba TomasVotruba changed the title tv skip non attribute [php 8.0] Check if class is an attribute in AnnotationToAttributeRector, to avoid non-attribute bug Jun 2, 2025
@TomasVotruba TomasVotruba force-pushed the tv-skip-non-attribute branch from 883079b to 9dc6f52 Compare June 2, 2025 13:16
@TomasVotruba TomasVotruba force-pushed the tv-skip-non-attribute branch from ce1b856 to 1ec4d22 Compare June 2, 2025 13:40
@TomasVotruba TomasVotruba enabled auto-merge (squash) June 2, 2025 13:47
@TomasVotruba TomasVotruba merged commit edd652b into main Jun 2, 2025
45 checks passed
@TomasVotruba TomasVotruba deleted the tv-skip-non-attribute branch June 2, 2025 13:47
@@ -1,37 +0,0 @@
<?php
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I will look up on these removed tests and make it work again, ensuring no regression in the future, it seems it missing target stubs in exact source file

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@github-actions
Copy link
Contributor

This pull request has been automatically locked because it has been closed for 150 days. Please open a new PR if you want to continue the work.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Nov 28, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants