Skip to content

remove: abandonedl package aminas-text#42

Open
shashi011 wants to merge 1 commit intolaminas:2.19.xfrom
shashi011:remove-laminas-text
Open

remove: abandonedl package aminas-text#42
shashi011 wants to merge 1 commit intolaminas:2.19.xfrom
shashi011:remove-laminas-text

Conversation

@shashi011
Copy link
Copy Markdown

@shashi011 shashi011 commented Jan 7, 2025

Q A
Documentation yes/no
Bugfix yes/no
BC Break yes/no
New Feature yes/no
RFC yes/no
QA yes/no

Description

This package currently depends on laminas-text, which has been abandoned. To address this issue, this pull request proposes removing the dependency on laminas-text.

@shashi011 shashi011 closed this Jan 7, 2025
@froschdesign
Copy link
Copy Markdown
Member

@shashi011
As you can see, removing it from the Composer configuration file does not work, because all dependent tests and the captcha itself will no longer work. This also represents a break in backward compatibility.
The Figlet captcha should be deprecated. See #41

@shashi011 shashi011 deleted the remove-laminas-text branch January 7, 2025 10:33
@shashi011 shashi011 restored the remove-laminas-text branch January 7, 2025 10:53
@shashi011 shashi011 reopened this Jan 7, 2025
@marclaporte
Copy link
Copy Markdown

@shashi011 Can you please try again and remove relevant parts of the code so we can release 3.0.0? #51

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.

3 participants