Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support for composer 2.3.0 #46

Merged
merged 18 commits into from Mar 22, 2022
Merged

Conversation

boesing
Copy link
Member

@boesing boesing commented Mar 16, 2022

Q A
BC Break no
New Feature yes

Description

Tests are still failing due to composer/composer#10622
This also re-enables unit tests for composer v1 which were removed in v2.6.0

This also re-enables tests for composer v1 which were dropped when migrating to PHP 8.1

Signed-off-by: Maximilian Bösing <2189546+boesing@users.noreply.github.com>
Signed-off-by: Maximilian Bösing <2189546+boesing@users.noreply.github.com>
@boesing boesing added this to the 2.7.0 milestone Mar 16, 2022
Signed-off-by: Maximilian Bösing <2189546+boesing@users.noreply.github.com>
Signed-off-by: Maximilian Bösing <2189546+boesing@users.noreply.github.com>
Signed-off-by: Maximilian Bösing <2189546+boesing@users.noreply.github.com>
Signed-off-by: Maximilian Bösing <2189546+boesing@users.noreply.github.com>
We do not need composer v1 to run tests on composer v1. We can install composer v1 as `require-dev` using composer v2. Afterwards, unit tests are able to detect composer v1 by applying the appropriate checks.

Signed-off-by: Maximilian Bösing <2189546+boesing@users.noreply.github.com>
Signed-off-by: Maximilian Bösing <2189546+boesing@users.noreply.github.com>
…uire`

Signed-off-by: Maximilian Bösing <2189546+boesing@users.noreply.github.com>
…json`

By adding an empty object to the config, we ensure that the default behavior of ignoring platform requirements for PHP 8 is disabled.

Signed-off-by: Maximilian Bösing <2189546+boesing@users.noreply.github.com>
…us implementation

Signed-off-by: Maximilian Bösing <2189546+boesing@users.noreply.github.com>
@boesing boesing marked this pull request as draft March 16, 2022 16:10
@Ocramius
Copy link
Member

Can we kill composer v1 support, perhaps? :|

@boesing
Copy link
Member Author

boesing commented Mar 16, 2022

I would prefer drop composer v1 when composer stops supporting it itself or when the maintenance amount gets too big. but as of now, maintaining v1 is not the problem.
But thats only my personal opinion here, if we decide to do so from the org perspective, I'd be fine with dropping it as well.

@Ocramius
Copy link
Member

Ok, so I guess it's just about getting this build back to green then? I'm perfectly OK with keeping composer/composer:^1 support, if it's not the blocker for this adjustment :)

This also restores tests on older composer v2 versions

Signed-off-by: Maximilian Bösing <2189546+boesing@users.noreply.github.com>
Signed-off-by: Maximilian Bösing <2189546+boesing@users.noreply.github.com>
Signed-off-by: Maximilian Bösing <2189546+boesing@users.noreply.github.com>
This resolves type incompatibilities with latest mocking changes.

Signed-off-by: Maximilian Bösing <2189546+boesing@users.noreply.github.com>
@boesing boesing marked this pull request as ready for review March 22, 2022 11:20
@boesing boesing requested a review from Ocramius March 22, 2022 11:20
.laminas-ci.json Show resolved Hide resolved
composer.json Outdated Show resolved Hide resolved
src/ComponentInstaller.php Show resolved Hide resolved
…n a string

Signed-off-by: Maximilian Bösing <2189546+boesing@users.noreply.github.com>
Signed-off-by: Maximilian Bösing <2189546+boesing@users.noreply.github.com>
src/ComponentInstaller.php Outdated Show resolved Hide resolved
Signed-off-by: Maximilian Bösing <2189546+boesing@users.noreply.github.com>
@Ocramius Ocramius self-assigned this Mar 22, 2022
Copy link
Member

@Ocramius Ocramius left a comment

Choose a reason for hiding this comment

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

LGTM 🚢

@Ocramius Ocramius merged commit 8597c23 into laminas:2.7.x Mar 22, 2022
@Ocramius
Copy link
Member

Thanks @boesing - cutting a release!

@boesing boesing deleted the feature/composer-2.3 branch March 22, 2022 15:32
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.

None yet

2 participants