Skip to content
This repository has been archived by the owner on Dec 3, 2023. It is now read-only.

Drop dependency on Composer #2406

Merged
merged 4 commits into from
Oct 25, 2020
Merged

Drop dependency on Composer #2406

merged 4 commits into from
Oct 25, 2020

Conversation

TomasVotruba
Copy link
Member

No description provided.

@TomasVotruba TomasVotruba force-pushed the ci-fix branch 2 times, most recently from 2778895 to 1322d22 Compare October 24, 2020 21:57
@TomasVotruba TomasVotruba changed the title make ci pass for new composer Drop dependency on Composer Oct 24, 2020
@TomasVotruba TomasVotruba force-pushed the ci-fix branch 11 times, most recently from 7ca038c to 2ff5470 Compare October 25, 2020 00:02
@@ -55,6 +55,8 @@ public function decoratePackageComposerJsonWithPackageSymlinks(
'options' => [
'symlink' => false,
],
// since composer 2.0 - see https://getcomposer.org/doc/articles/repository-priorities.md#default-behavior
'canonical' => false,
Copy link
Member Author

Choose a reason for hiding this comment

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

@samsonasik In the end it took more refactoring to find the bug.
This was the line, that was one of important fixes with new Composer 2.

This behavior changed: https://getcomposer.org/doc/articles/repository-priorities.md#default-behavior

Copy link
Collaborator

Choose a reason for hiding this comment

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

@TomasVotruba thank you

@TomasVotruba TomasVotruba merged commit 09cdb52 into master Oct 25, 2020
@TomasVotruba TomasVotruba deleted the ci-fix branch October 25, 2020 10:35
@deprecated-packages deprecated-packages locked as resolved and limited conversation to collaborators Nov 14, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants