10000 Redundant PHPdoc line · Issue #52932 · symfony/symfony · GitHub
[go: up one dir, main page]
Skip to content

Redundant PHPdoc line #52932

New issue

Have a question about 8000 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
Closed
rarila opened this issue Dec 7, 2023 · 0 comments
Closed

Redundant PHPdoc line #52932

rarila opened this issue Dec 7, 2023 · 0 comments
Labels
Bug Status: Needs Review

Comments

@rarila
Copy link
rarila commented Dec 7, 2023

Symfony version(s) affected

6.3 - 7.1

Description

First line of PHPdoc is redundant:

symfony/src/Symfony/Component/DependencyInjection/EnvVarProcessorInterface.php

Lines 26 to 27 in c71348a

* @param string $prefix The namespace of the variable
* @param string $prefix The namespace of the variable; when the empty string is passed, null values should be kept as is

Makes Psalm fail getting the Closure signature, which of course is a Psalm problem. Anyway, first line should be removed.

How to reproduce

—

Possible Solution

Removing the first line :-)

Additional Context

No response
@rarila rarila added the Bug label Dec 7, 2023
@StefanosPs StefanosPs mentioned this issue Dec 7, 2023
Merged
OskarStark added a commit that referenced this issue Dec 8, 2023
@OskarStark
minor #52933 [DependencyInjection] Remove redundant PHPdoc line (Stef…
96421d3 
…anosPs)

This PR was merged into the 6.3 branch.

Discussion
----------

[DependencyInjection] Remove redundant PHPdoc line

| Q             | A
| ------------- | ---
| Branch?       | 6.3
| Bug fix?      | yes
| New feature?  | no
| Deprecations? | no
| Issues        | Fix #52932
| License       | MIT

Just remove a Redundant PHPdoc line

Commits
-------

d6d169b Remove redundant PHPdoc line
@xabbuh xabbuh closed this as completed Dec 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Bug Status: Needs Review
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@xabbuh @rarila @carsonbot
0