Skip to content

Conversation

@Pytal
Copy link
Member

@Pytal Pytal commented Apr 5, 2022

OCP package christophwurst/nextcloud is set to ^20.0.0, the minimum supported version of the app

An OCP version matrix is also added to lint against newer versions

Signed-off-by: Christopher Ng <chrng8@gmail.com>
Pytal added 2 commits April 5, 2022 01:52
Signed-off-by: Christopher Ng <chrng8@gmail.com>
Signed-off-by: Christopher Ng <chrng8@gmail.com>
@Pytal
Copy link
Member Author

Pytal commented Apr 5, 2022

@nickvergessen @CarlSchwan any ideas on the errors?

Signed-off-by: Joas Schilling <coding@schilljs.com>
Signed-off-by: Joas Schilling <coding@schilljs.com>
@Pytal
Copy link
Member Author

Pytal commented Apr 6, 2022

Some errors may be fixed by ChristophWurst/nextcloud_composer#18

@Pytal
Copy link
Member Author

Pytal commented Apr 6, 2022

For this error

christophwurst/nextcloud v22.1.1 requires psr/container ^1.0 -> found psr/container[1.0.0, 1.1.0, 1.1.1, 1.1.2] but the package is fixed to 2.0.2 (lock file version) by a partial update and that version does not match.

The dependency tree of this app https://github.com/nextcloud/files_antivirus shows nextcloud/coding-standard:^0.4.0 -> friendsofphp/php-cs-fixer:^2.17 -> ... -> psr/container:^2.0

But in the server 3rdparty repo the latest is psr/container v1.1.1 https://github.com/nextcloud/3rdparty/tree/d80ec1fa2dad1c3ede272583e3c4f1f77f40141b/psr/container so we can't proceed with ChristophWurst/nextcloud_composer#18 (comment)

Not sure how to resolve these conflicts, any ideas @CarlSchwan?

@PVince81 PVince81 mentioned this pull request Apr 8, 2022
@PVince81
Copy link
Member

@ChristophWurst can you help with the above question about dependency issues ?

@Pytal Pytal removed their assignment Apr 12, 2022
Signed-off-by: Robin Appelman <robin@icewind.nl>
@icewind1991 icewind1991 merged commit 0d63038 into master Apr 13, 2022
@icewind1991 icewind1991 deleted the fix/php7.3-ci branch April 13, 2022 12:20
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.

5 participants