Skip to content

Conversation

@VincentLanglet
Copy link
Contributor

Q A
Bug fix? no
New feature? no
BC breaks? no
Deprecations? no
Fixed tickets Fix #...
License MIT

Hi @sstok, while I appreciate you worked on SF8 support, #71
I feel like the support for SF 7 (and 6 ?) was dropped to fast.

Currently, when using Symfony 7.4 we have the choice between

  • Deprecations on 3.x branch because xmlLoader is used
  • A 4.x branch which fixed this deprecation but doesn't support 7.4

All the (Breaking ?) changes you made for the 3.0.0 versions are still combatible with Symfony 7 ! (and symfony 6 too I think)
So you could keep at least 7.4 support.

Thanks for your consideration.

@sstok
Copy link
Member

sstok commented Dec 17, 2025

Guess I have been a bit overzealous, technically the $options argument no longer works. But everything else is still compatible.

@sstok sstok merged commit 4c7fb28 into rollerworks:main Dec 17, 2025
4 checks passed
@sstok
Copy link
Member

sstok commented Dec 17, 2025

Thank you @VincentLanglet

@sstok
Copy link
Member

sstok commented Dec 17, 2025

Can you do the same for the https://github.com/rollerworks/PasswordStrengthBundle ? Thanks

@VincentLanglet
Copy link
Contributor Author

Can you do the same for the rollerworks/PasswordStrengthBundle ? Thanks

Sure, I opened rollerworks/PasswordStrengthBundle#121

Also what is your plan for the final 3.0.0 version ? :)

sstok added a commit to rollerworks/PasswordStrengthBundle that referenced this pull request Dec 17, 2025
This PR was merged into the 4.0-dev branch.

Discussion
----------

| Q             | A
| ------------- | ---
| Bug fix?      | no
| New feature?  | no
| BC breaks?    | no
| Deprecations? | no
| Fixed tickets | Fix #... <!-- prefix each issue number with "Fix #", no need to create an issue if none exist, explain below instead -->
| License       | MIT

Same as rollerworks/PasswordStrengthValidator#74

Commits
-------

bc5894b Keep support for SF 7
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants