-
-
Notifications
You must be signed in to change notification settings - Fork 31
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
Remove deprecated filters #102
Conversation
If features are removed then this must be documented in a migration guide. - Migration:
- "Migration from Version 2 to 3": v3/migration/v2-to-v3.md Example: https://docs.laminas.dev/laminas-form/v3/migration/v2-to-v3/ |
Is it best to revert docs changes here and move them to a |
@gsteel You are right, a revert and a separate pull request is the correct way. |
7c7e4aa
to
677b5e4
Compare
Signed-off-by: George Steel <[email protected]>
Signed-off-by: George Steel <[email protected]>
Signed-off-by: George Steel <[email protected]>
Signed-off-by: George Steel <[email protected]>
Instead of altering docs for the v2 series, new docs must be created and amended for the v3 series Signed-off-by: George Steel <[email protected]>
677b5e4
to
92bf299
Compare
Signed-off-by: George Steel <[email protected]>
Any chance of an approval on this one? Everything is green 🙏 |
Thanks @samsonasik |
Removes the deprecated filters:
Cut from #101, so this will need a rebase and a 3.0 branch once that's merged.
Removing the dev dependency on laminas-crypt clears the way for testing this lib on laminas-servicemanger 4.x