Skip to content

IBX-11507: Merged branch '4.6' into 5.0 - #102

Merged
Sztig merged 2 commits into
5.0from
merge/4.6-to-5.0
Jul 30, 2026
Merged

IBX-11507: Merged branch '4.6' into 5.0#102
Sztig merged 2 commits into
5.0from
merge/4.6-to-5.0

Conversation

@Sztig

@Sztig Sztig commented Jul 29, 2026

Copy link
Copy Markdown
Contributor

…th limited permissions (#100)

* added new listener and log channel to handle ArgumentsException logging

* refactored fix after review

* implemented tests for the ArgumentsExceptionListener

* fixed test for php7.4

* added phpunit components to the bundle

* refactored the test into phpunit

* implemented suggested changes
@Sztig Sztig added the Fast-forward merge PR should be merged in a fast-forward way label Jul 29, 2026
@Sztig
Sztig force-pushed the merge/4.6-to-5.0 branch from a9956e6 to f8ae9de Compare July 29, 2026 10:36
@Sztig
Sztig marked this pull request as ready for review July 29, 2026 10:37
@Sztig
Sztig requested a review from a team July 29, 2026 10:37
Comment thread src/lib/EventListener/ArgumentsExceptionListener.php Outdated
Comment thread tests/lib/EventListener/ArgumentsExceptionListenerTest.php Outdated
Comment thread composer.json
# Conflicts:
#	composer.json
@Sztig
Sztig force-pushed the merge/4.6-to-5.0 branch from f8ae9de to 0f30ba9 Compare July 29, 2026 11:15
@sonarqubecloud

Copy link
Copy Markdown

@konradoboza
konradoboza requested a review from bnowak July 29, 2026 12:08

@KamilSznajdrowicz KamilSznajdrowicz left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

QA Approved

@Sztig
Sztig merged commit 0f30ba9 into 5.0 Jul 30, 2026
11 checks passed
@Sztig
Sztig deleted the merge/4.6-to-5.0 branch July 30, 2026 11:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Fast-forward merge PR should be merged in a fast-forward way QA approved Ready for review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants