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.

Sign up for GitHub

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

Jump to bottom

ArgumentsAnalyzer - support PHP8.1 readonly #6224

Merged
merged 1 commit into from Jan 9, 2022
Merged

ArgumentsAnalyzer - support PHP8.1 readonly #6224

merged 1 commit into from Jan 9, 2022

Conversation

SpacePossum
Copy link
Contributor

closes #6221

@coveralls
Copy link

coveralls commented Jan 5, 2022 •
edited

Coverage Status

Coverage decreased (-0.01%) to 93.013% when pulling 8daa46e on SpacePossum:master_6221_bug into efeab76 on FriendsOfPHP:master.

@kubawerlos kubawerlos added the RTM Ready To Merge label Jan 7, 2022
@SpacePossum SpacePossum merged commit f4b0017 into PHP-CS-Fixer:master Jan 9, 2022
@SpacePossum SpacePossum removed the RTM Ready To Merge label Jan 9, 2022
@SpacePossum SpacePossum deleted the master_6221_bug branch January 9, 2022 19:39
@krzysztofrewak krzysztofrewak mentioned this pull request Jan 21, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jrmajor jrmajor jrmajor left review comments

@GrahamCampbell GrahamCampbell GrahamCampbell approved these changes

@kubawerlos kubawerlos kubawerlos approved these changes

@krzysztofrewak krzysztofrewak krzysztofrewak approved these changes

Assignees
No one assigned
Labels
kind/bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

nullable_type_declaration_for_default_null_value fails for nullable readonly parameter in promoted constructor
6 participants
@SpacePossum @coveralls @GrahamCampbell @kubawerlos @krzysztofrewak @jrmajor