Skip to content

Fix invalid interface mention in security.rst #16700

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

Merged
merged 2 commits into from
Apr 21, 2022
Merged

Conversation

ttibensky
Copy link
Contributor

@carsonbot carsonbot added this to the 5.4 milestone Apr 10, 2022
@javiereguiluz javiereguiluz requested a review from wouterj April 12, 2022 15:51
@wouterj wouterj merged commit 03e9b7b into symfony:5.4 Apr 21, 2022
@wouterj
Copy link
Member

wouterj commented Apr 21, 2022

Thank you @ttibensky, you are completely correct.

I've slightly modified the change to no longer mention the interface. It is deprecated since PHP 8.1 (the latest PHP version), so I think it's better to remove the mention. We now talk about the method names, which are future proof.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants