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

Refactor AnnotationExtractor fomr FileDiff value object to standalone RectorsChangelogResolver service #6088

Merged
merged 2 commits into from
Apr 10, 2021

Conversation

TomasVotruba
Copy link
Member

@TomasVotruba TomasVotruba commented Apr 10, 2021

Small follow up to #6073

Looking at it closre, the FileDiff should be only dummy value object not knowing about formatters and output. This should separate the annotation logic out to a service

@TomasVotruba TomasVotruba force-pushed the improve-annot-resolver branch 2 times, most recently from 7633ccf to ab6b960 Compare April 10, 2021 16:06
@TomasVotruba TomasVotruba changed the title refactor AnnotationExtractor fomr FileDiff value object to standalone RectorsChangelogResolver service Refactor AnnotationExtractor fomr FileDiff value object to standalone RectorsChangelogResolver service Apr 10, 2021
@TomasVotruba TomasVotruba force-pushed the improve-annot-resolver branch 2 times, most recently from fee55a9 to 2d9442b Compare April 10, 2021 16:09
@TomasVotruba TomasVotruba enabled auto-merge (squash) April 10, 2021 16:09
@TomasVotruba TomasVotruba force-pushed the improve-annot-resolver branch from 2d9442b to 5fa7e15 Compare April 10, 2021 16:10
@TomasVotruba TomasVotruba merged commit f82a8dc into main Apr 10, 2021
@TomasVotruba TomasVotruba deleted the improve-annot-resolver branch April 10, 2021 16:22
TomasVotruba added a commit that referenced this pull request Jun 30, 2024
rectorphp/rector-src@9fa1da9 Add few union type fixtures to ReturnTypeFromStrictTypedCallRector (#6088)
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.

1 participant