Skip to content

feat: bump laminas diagnostics to version 1.27#321

Closed
shakaran wants to merge 2 commits intoliip:2.xfrom
shakaran:feat/bump-laminas-diagnostics
Closed

feat: bump laminas diagnostics to version 1.27#321
shakaran wants to merge 2 commits intoliip:2.xfrom
shakaran:feat/bump-laminas-diagnostics

Conversation

@shakaran
Copy link
Copy Markdown

No description provided.

composer.json Outdated
"php": "^8.1",
"symfony/framework-bundle": "^6.4|^7.0",
"laminas/laminas-diagnostics": "^1.9"
"laminas/laminas-diagnostics": "^1.27"
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Instead of this, I think we need:

"enlightn/security-checker": "^1.11|^2.0",

Copy link
Copy Markdown
Author

@shakaran shakaran Jun 25, 2025

Choose a reason for hiding this comment

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

updated at 9147957

Copy link
Copy Markdown
Member

@dbu dbu left a comment

Choose a reason for hiding this comment

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

allowed security checker in #330

"php": "^8.1",
"symfony/framework-bundle": "^6.4|^7.0",
"laminas/laminas-diagnostics": "^1.9"
"laminas/laminas-diagnostics": "^1.9 || ^1.27"
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.

this is odd, ^1.9 includes ^1.27, so the change does not do anything.

"doctrine/migrations": "^2.0 || ^3.0",
"doctrine/persistence": "^1.3.3 || ^2.0 || ^3.0",
"enlightn/security-checker": "^1.11",
"enlightn/security-checker": "^1.11 || ^2.0",
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.

i did this in #330

@dbu dbu closed this Mar 23, 2026
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