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

EZP-32307: Fixed check version stability #86

Merged
merged 1 commit into from
Jan 28, 2021

Conversation

ciastektk
Copy link
Contributor

@ciastektk ciastektk commented Jan 22, 2021

https://issues.ibexa.co/browse/EZP-32307

This PR provides implemented VersionStabilityChecker for displaying valid version stability. Since Ibexa/website-skeleton (https://github.com/ibexa/website-skeleton/blob/main/composer.json#L6) provides minimum-stability set to dev all versions are marked as ‘dev’. Now version stability is based on ibexa/oss version because this package is installed for all Ibexa Platform versions. For BC break reason old check based on composer.lock minimum stability is set as callback in case when Ibexa/oss is not found.

@ciastektk ciastektk force-pushed the fixed-check-version-stability branch from 7cc4877 to a3e49f1 Compare January 22, 2021 13:35
@ciastektk ciastektk force-pushed the fixed-check-version-stability branch 2 times, most recently from cbd6312 to 9b2c36a Compare January 25, 2021 13:18
@ciastektk ciastektk marked this pull request as ready for review January 25, 2021 13:18
@ciastektk ciastektk force-pushed the fixed-check-version-stability branch 4 times, most recently from 6664a74 to 3bb8d36 Compare January 26, 2021 13:38
Copy link
Member

@alongosz alongosz left a comment

Choose a reason for hiding this comment

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

Good responsibility separation!

$composerCollectorNotFound = new JsonComposerLockSystemInfoCollector(__DIR__ . '/_fixtures/snafu.lock', __DIR__ . '/_fixtures/composer.json');
$composerCollectorNotFound = new JsonComposerLockSystemInfoCollector(
$this->composerInstalledVersions,
__DIR__ . '/_fixtures/snafu.lock',
Copy link
Member

Choose a reason for hiding this comment

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

😂

@ciastektk ciastektk force-pushed the fixed-check-version-stability branch from 3bb8d36 to 952f490 Compare January 26, 2021 13:48
@ciastektk ciastektk changed the title Fixed check version stability EZP-32307: Fixed check version stability Jan 26, 2021
@tomaszszopinski tomaszszopinski self-assigned this Jan 27, 2021
Copy link

@tomaszszopinski tomaszszopinski left a comment

Choose a reason for hiding this comment

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

QA approved on Ibexa oss/content/ezp/commerce stable 3.3@rc with patch.

@lserwatka lserwatka merged commit 8df047b into ezsystems:master Jan 28, 2021
@ciastektk ciastektk deleted the fixed-check-version-stability branch January 28, 2021 13:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
4 participants