Skip to content

Conversation

@backportbot-nextcloud
Copy link

@backportbot-nextcloud backportbot-nextcloud bot commented May 17, 2022

⚠️ This backport had conflicts and is incomplete ⚠️

backport of #31623

/** @var string accounts table name */
private $table = 'accounts';
/** accounts table name */
private string $table = 'accounts';
Copy link
Member

Choose a reason for hiding this comment

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

Not php 7.3 compatible


/** @var AccountManager */
private $accountManager;
private AccountManager $accountManager;
Copy link
Member

Choose a reason for hiding this comment

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

Not php 7.3 compatible

@szaimen
Copy link
Contributor

szaimen commented May 17, 2022

I suppose if we merge this we would need a new minor release?
cc @PVince81

@tcitworld
Copy link
Member

Does it really need backports? I don't think so.

@szaimen
Copy link
Contributor

szaimen commented Jun 9, 2022

Does it really need backports? I don't think so.

Cc @PVince81 @AndyScherzinger

@blizzz
Copy link
Member

blizzz commented Jun 10, 2022

At least moving to 23.0.7 for now for it is not even in review state.

@blizzz blizzz mentioned this pull request Jun 17, 2022
4 tasks
@szaimen
Copy link
Contributor

szaimen commented Jun 25, 2022

So seems like we will not merge this? then lets close it.
cc @PVince81 @AndyScherzinger

@AndyScherzinger
Copy link
Member

If we can make it work for 23 with little effort (the php 7.3 compatibility) I'd say do it, else let's skip 23 but def. do 24

This was referenced Jul 5, 2022
@blizzz blizzz mentioned this pull request Jul 18, 2022
@blizzz
Copy link
Member

blizzz commented Jul 18, 2022

moving to 23.0.8…

@blizzz blizzz mentioned this pull request Aug 1, 2022
@blizzz
Copy link
Member

blizzz commented Aug 1, 2022

@PVince81 what do we do here?

@PVince81
Copy link
Member

PVince81 commented Aug 1, 2022

let's not backport

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

Labels

2. developing Work in progress enhancement feature: users and groups php Pull requests that update Php code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants