Skip to content

Conversation

@ArtificialOwl
Copy link
Member

backport of #39574

@ArtificialOwl ArtificialOwl force-pushed the backport/39574/39574-stable27 branch from 5ae9081 to 19f7fa8 Compare September 22, 2023 13:50
@ArtificialOwl
Copy link
Member Author

@provokateurin do you think #40575 will fix psalm analysis ? should we cherrypick ?

@provokateurin
Copy link
Member

No my PR won't fix it. You need to remove all the new @return annotations since those are not supported lower than master/28.

@ArtificialOwl ArtificialOwl force-pushed the backport/39574/39574-stable27 branch 2 times, most recently from ef4d72e to 1af7102 Compare September 22, 2023 15:20
@provokateurin
Copy link
Member

I think you should remove the annotation instead of supressing it

Signed-off-by: Maxence Lange <[email protected]>
@ArtificialOwl ArtificialOwl force-pushed the backport/39574/39574-stable27 branch from 1af7102 to 0fac750 Compare September 22, 2023 18:26
@ArtificialOwl ArtificialOwl added this to the Nextcloud 27.1.2 milestone Sep 22, 2023
@ArtificialOwl ArtificialOwl added 3. to review Waiting for reviews 4. to release Ready to be released and/or waiting for tests to finish labels Sep 22, 2023
@ArtificialOwl ArtificialOwl requested review from a team, icewind1991, nfebe and sorbaugh and removed request for a team September 22, 2023 20:40
@ArtificialOwl ArtificialOwl merged commit b06bff7 into stable27 Sep 23, 2023
@ArtificialOwl ArtificialOwl deleted the backport/39574/39574-stable27 branch September 23, 2023 15:08
@blizzz blizzz mentioned this pull request Sep 26, 2023
@glpatcern
Copy link

For info, NC 27.1.2 was deployed at PSNC in Poland, a site of the ScienceMesh (the infra where the initial report #39001 originated), and I can confirm basic sharing functionally works as expected. Thanks for the effort!

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

Labels

3. to review Waiting for reviews 4. to release Ready to be released and/or waiting for tests to finish

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

6 participants