Skip to content

Conversation

@come-nc
Copy link
Contributor

@come-nc come-nc commented Nov 29, 2021

Signed-off-by: Côme Chilliet come.chilliet@nextcloud.com

Needed for PHP 8.1 compatibility

Replaces #935

Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
@come-nc come-nc force-pushed the fix/bump-christophwurst/id3parser-to-0.1.4 branch from 6a7b6be to 1f50b25 Compare November 30, 2021 09:29
@come-nc come-nc merged commit 26d99dc into master Nov 30, 2021
@come-nc come-nc deleted the fix/bump-christophwurst/id3parser-to-0.1.4 branch November 30, 2021 09:33
@pachulo
Copy link

pachulo commented Dec 8, 2021

Hi! Could this be backported to stable22 and stable21?
Thanks!

@ChristophWurst
Copy link
Member

What would be the benefits of the backport?

@pachulo
Copy link

pachulo commented Dec 9, 2021

Without the backport https://github.com/nextcloud/previewgenerator throws errors when using PHP 8.x, which is supported by nextcloud 21 & 22.

More info: nextcloud/previewgenerator#249

@pachulo
Copy link

pachulo commented Dec 19, 2021

so, how can this be done @ChristophWurst ?

@ChristophWurst
Copy link
Member

  • Check out the stable branch, e.g. stable21
  • Change composer.json
  • Run composer update christophwurst/id3parser`
  • Branch off into a new branch
  • Commit
  • Push the branch
  • Open a pull request

✌️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants