Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Commit a8c5911

Browse filesBrowse files
committed
minor #14916 Update conventions page to show correct function (dayallnash)
This PR was merged into the 5.2 branch. Discussion ---------- Update conventions page to show correct function * Updated conventions page to say trigger_deprecation instead of trigger_error Commits ------- 1bb5c08 Update conventions page to show correct function
2 parents 8d4c9ba + 1bb5c08 commit a8c5911
Copy full SHA for a8c5911

File tree

1 file changed

+1
-1
lines changed
Filter options

1 file changed

+1
-1
lines changed

‎contributing/code/conventions.rst

Copy file name to clipboardExpand all lines: contributing/code/conventions.rst
+1-1Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -167,7 +167,7 @@ A deprecation must also be triggered to help people with the migration
167167

168168
trigger_deprecation('symfony/package-name', '5.1', 'The "%s" class is deprecated, use "%s" instead.', Deprecated::class, Replacement::class);
169169

170-
When deprecating a whole class the ``trigger_error()`` call should be placed
170+
When deprecating a whole class the ``trigger_deprecation()`` call should be placed
171171
after the use declarations, like in this example from `ServiceRouterLoader`_::
172172

173173
namespace Symfony\Component\Routing\Loader\DependencyInjection;

0 commit comments

Comments
0 (0)
Morty Proxy This is a proxified and sanitized view of the page, visit original site.