feat: Introduce config option for standardized null in typehints #7860
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR introduces a new configuration option
force_nullable_for_default_null
tonullable_type_declaration_for_default_null_value
rule, aimed at enhancing the consistency and clarity of typehints across our codebase.By automatically converting optional types to explicit union types with
null
, we enable a more standardized approach to handling nullable type declarations.Key Benefits:
?Type
toType|null
makes the nullable aspect of the typehints explicit, improving readability, and comprehension for developers.Implementation Details:
The configuration option
force_nullable_for_default_null
can be toggled to automatically modify the typehints during the PHPCSFixer fix process. When enabled, it ensures that all parameters with a defaultnull
value explicitly includenull
in their type declaration, transforming?Type
hints toType|null
for clearer and more explicit code.