Skip to content

Allow laminas/laminas-validator 3.0 - #90

Merged
gsteel merged 1 commit into
laminas:2.21.xfrom
marcelthole:2.21.x
Dec 4, 2024
Merged

Allow laminas/laminas-validator 3.0#90
gsteel merged 1 commit into
laminas:2.21.xfrom
marcelthole:2.21.x

Conversation

@marcelthole

Copy link
Copy Markdown
Contributor

See #83

i accidently updated the source branch and closed the linked PR because there wasn't any diff anymore :/

@gsteel

gsteel commented Dec 2, 2024

Copy link
Copy Markdown
Member

v3.0 is not being installed at all: https://github.com/laminas/laminas-http/actions/runs/12123750099/job/33799937150#step:3:78

Can I suggest you try a composer why-not laminas/laminas-validator 3.0?

@froschdesign

Copy link
Copy Markdown
Member

@gsteel

laminas/laminas-uri       2.12.0     requires         laminas/laminas-validator (^2.39)                        
laminas/laminas-validator 3.0.0      requires         laminas/laminas-servicemanager (^4.1.0)                  
laminas/laminas-http      2.21.x-dev does not require laminas/laminas-servicemanager (but 3.23.0 is installed)

@gsteel

gsteel commented Dec 2, 2024

Copy link
Copy Markdown
Member

Aye, so laminas-uri is the blocker - we'll have to update it with v3.0 support first

@marcelthole

Copy link
Copy Markdown
Contributor Author

I created the PR for laminas-uri to support laminas-validator 3 - laminas/laminas-uri#24

@gsteel

gsteel commented Dec 4, 2024

Copy link
Copy Markdown
Member

Now URI is released, can you please refresh composer.lock and re-push so we can see CI installing v3?

Signed-off-by: Marcel Thole <marcel@marcelthole.de>
@marcelthole

Copy link
Copy Markdown
Contributor Author
  • Downloading laminas/laminas-validator (3.0.1)

Yes, version 3.0.1 could be installed

@froschdesign froschdesign added this to the 2.21.0 milestone Dec 4, 2024

@gsteel gsteel left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you @marcelthole :)

@gsteel gsteel self-assigned this Dec 4, 2024
@gsteel
gsteel merged commit 6440d57 into laminas:2.21.x Dec 4, 2024
@marcelthole
marcelthole deleted the 2.21.x branch December 4, 2024 10:54
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