Skip to content

Adapt to ipl-stdlib strict typing#187

Merged
lippserd merged 2 commits intomainfrom
modernize-code
Mar 24, 2026
Merged

Adapt to ipl-stdlib strict typing#187
lippserd merged 2 commits intomainfrom
modernize-code

Conversation

@BastianLedererIcinga
Copy link
Contributor

@BastianLedererIcinga BastianLedererIcinga commented Mar 2, 2026

Enforce strict type declarations from ipl-stdlib, update implementing
classes for signature compliance, and bump ipl/* versions in
composer.json.

requires Icinga/ipl-stdlib#63

@sukhwinder33445
Copy link
Contributor

You have pushed some old commits, please cleanup.

@BastianLedererIcinga BastianLedererIcinga changed the base branch from main to support-php-85 March 3, 2026 06:48
sukhwinder33445
sukhwinder33445 previously approved these changes Mar 3, 2026
Base automatically changed from support-php-85 to main March 5, 2026 12:37
@lippserd lippserd dismissed sukhwinder33445’s stale review March 5, 2026 12:37

The base branch was changed.

@BastianLedererIcinga BastianLedererIcinga changed the title Modernize code Ipl forward compatibility Mar 6, 2026
@BastianLedererIcinga BastianLedererIcinga changed the title Ipl forward compatibility ipl-stdlib forward compatibility Mar 6, 2026
@lippserd lippserd changed the title ipl-stdlib forward compatibility Adapt to ipl-stdlib strict typing Mar 19, 2026
BastianLedererIcinga and others added 2 commits March 24, 2026 22:32
`ipl-stdlib` introduces strict type declarations, requiring all classes
implementing its interfaces to match the exact parameter and return types
defined there. Updated affected classes and bumped the required `ipl/*`
versions in `composer.json` accordingly.

Co-authored-by: Eric Lippmann <eric.lippmann@icinga.com>
@lippserd lippserd merged commit 58468f5 into main Mar 24, 2026
13 checks passed
@lippserd lippserd deleted the modernize-code branch March 24, 2026 21:35
@lippserd lippserd added this to the v0.10.0 milestone Mar 24, 2026
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.

3 participants