Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@Drkilla
Copy link

@Drkilla Drkilla commented May 27, 2025

Q A
Branch? 7.4
Bug fix? no
New feature? yes
Deprecations? no
Issues [Validator] Add option to allow ANY protocol in Assert\Url constraint
License MIT

@carsonbot
Copy link

Hey!

I see that this is your first PR. That is great! Welcome!

Symfony has a contribution guide which I suggest you to read.

In short:

  • Always add tests
  • Keep backward compatibility (see https://symfony.com/bc).
  • Bug fixes must be submitted against the lowest maintained branch where they apply (see https://symfony.com/releases)
  • Features and deprecations must be submitted against the 7.4 branch.

Review the GitHub status checks of your pull request and try to solve the reported issues. If some tests are failing, try to see if they are failing because of this change.

When two Symfony core team members approve this change, it will be merged and you will become an official Symfony contributor!
If this PR is merged in a lower version branch, it will be merged up to all maintained branches within a few days.

I am going to sit back now and wait for the reviews.

Cheers!

Carsonbot

@OskarStark OskarStark changed the title [Validator] Add option to allow ANY protocol in Assert\Url constraint [Validator] Add option to allow ANY protocol in Assert\Url constraint Jul 7, 2025
@Drkilla Drkilla force-pushed the feature/url-validator-allow-any-protocol branch from b68021b to ed1178c Compare July 7, 2025 21:26
@fabpot fabpot force-pushed the feature/url-validator-allow-any-protocol branch from ed1178c to d8b7f62 Compare September 22, 2025 05:52
@fabpot
Copy link
Member

fabpot commented Sep 22, 2025

Thank you @Drkilla.

@fabpot fabpot merged commit 88adaa6 into symfony:7.4 Sep 22, 2025
8 of 12 checks passed
nicolas-grekas added a commit that referenced this pull request Sep 24, 2025
…(xabbuh)

This PR was merged into the 7.4 branch.

Discussion
----------

[Validator] allow the asterisk to be passed as a string

| Q             | A
| ------------- | ---
| Branch?       | 7.4
| Bug fix?      | no
| New feature?  | no
| Deprecations? | no
| Issues        |
| License       | MIT

making #60561 easier to use

Commits
-------

a91e0ce allow protocols to be passed as a string
This was referenced Oct 27, 2025
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.

4 participants