Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork9.7k
Commitf2ef55c
committed
feature#42337 [Validator] Remove internal from
This PR was merged into the 5.4 branch.Discussion----------[Validator] Remove internal from `ConstraintViolationAssertion`| Q | A| ------------- | ---| Branch? | 5.4| Bug fix? | no| New feature? | no| Deprecations? | no| Tickets |Fix#42310| License | MIT| Doc PR | symfony/symfony-docs#... <!-- required for new features --><!--Replace this notice by a short README for your feature/bugfix. This will help peopleunderstand your PR and can be used as a start for the documentation.Additionally (seehttps://symfony.com/releases): - Always add tests and ensure they pass. - Never break backward compatibility (seehttps://symfony.com/bc). - Bug fixes must be submitted against the lowest maintained branch where they apply (lowest branches are regularly merged to upper ones so they get the fixes too.) - Features and deprecations must be submitted against branch 5.x. - Changelog entry should followhttps://symfony.com/doc/current/contributing/code/conventions.html#writing-a-changelog-entry-->As requested on the issue, I made this class final instead of internal and the construct method is internal.Commits-------97949ea Remove internal from ConstraintViolationAssertionConstraintViolationAssertion (jordisala1991)File tree
1 file changed
+4
-4
lines changed- src/Symfony/Component/Validator/Test
1 file changed
+4
-4
lines changedLines changed: 4 additions & 4 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
293 | 293 | | |
294 | 294 | | |
295 | 295 | | |
296 | | - | |
297 | | - | |
298 | | - | |
299 | | - | |
| 296 | + | |
300 | 297 | | |
301 | 298 | | |
302 | 299 | | |
| |||
317 | 314 | | |
318 | 315 | | |
319 | 316 | | |
| 317 | + | |
| 318 | + | |
| 319 | + | |
320 | 320 | | |
321 | 321 | | |
322 | 322 | | |
| |||
0 commit comments
Comments
(0)