Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[stable27] The almighty text processing PR #39776

Merged
merged 111 commits into from
Aug 9, 2023
Merged

Conversation

marcelklehr
Copy link
Member

@marcelklehr marcelklehr commented Aug 9, 2023

Contains backports of

Checklist

Signed-off-by: Marcel Klehr <[email protected]>
(cherry picked from commit 70d5bf7)
Signed-off-by: Marcel Klehr <[email protected]>
(cherry picked from commit 01dd1a8)
Signed-off-by: Marcel Klehr <[email protected]>
(cherry picked from commit 6fc4cb6)
Signed-off-by: Marcel Klehr <[email protected]>
(cherry picked from commit 82d3b00)
Signed-off-by: Marcel Klehr <[email protected]>
(cherry picked from commit 9e9fc1d)
Signed-off-by: Marcel Klehr <[email protected]>
(cherry picked from commit 9d5717d)
Signed-off-by: Marcel Klehr <[email protected]>

(cherry picked from commit 795b097)
Signed-off-by: Marcel Klehr <[email protected]>
(cherry picked from commit b6941aa)
Signed-off-by: Marcel Klehr <[email protected]>
(cherry picked from commit b8a9f08)
Signed-off-by: Marcel Klehr <[email protected]>
(cherry picked from commit b6a95e3)
Signed-off-by: Marcel Klehr <[email protected]>
(cherry picked from commit 9935034)
Signed-off-by: Marcel Klehr <[email protected]>
(cherry picked from commit fac83ce)
Signed-off-by: Marcel Klehr <[email protected]>
(cherry picked from commit 75d7af8)
Signed-off-by: Marcel Klehr <[email protected]>
(cherry picked from commit 72ea761)
Co-authored-by: Julien Veyssier <[email protected]>
Signed-off-by: Marcel Klehr <[email protected]>
(cherry picked from commit 249dd5c)
Signed-off-by: Marcel Klehr <[email protected]>
(cherry picked from commit fb4de16)
Signed-off-by: Marcel Klehr <[email protected]>
(cherry picked from commit 5bc6180)
Co-authored-by: Daniel <[email protected]>
Signed-off-by: Marcel Klehr <[email protected]>
(cherry picked from commit fb55afc)
Co-authored-by: Daniel <[email protected]>
Signed-off-by: Marcel Klehr <[email protected]>
(cherry picked from commit 8e4aa92)
Signed-off-by: Marcel Klehr <[email protected]>
(cherry picked from commit 906e9b7)
Signed-off-by: Marcel Klehr <[email protected]>
(cherry picked from commit 83db23e)
Signed-off-by: Marcel Klehr <[email protected]>
(cherry picked from commit a7cd6bf)
Signed-off-by: Marcel Klehr <[email protected]>
(cherry picked from commit b00a9a6)
Signed-off-by: Marcel Klehr <[email protected]>
(cherry picked from commit 94fcf88)
Signed-off-by: Marcel Klehr <[email protected]>
(cherry picked from commit 1d3661d)
Signed-off-by: Marcel Klehr <[email protected]>
(cherry picked from commit cb0f918)
Signed-off-by: Marcel Klehr <[email protected]>
(cherry picked from commit 1623ad9)
Signed-off-by: Marcel Klehr <[email protected]>
(cherry picked from commit 0a94525)
…zed types

Signed-off-by: Marcel Klehr <[email protected]>
(cherry picked from commit 27e1c86)
@julien-nc julien-nc force-pushed the backport/39680/stable27 branch 3 times, most recently from c4c3fe7 to 19fbce6 Compare August 9, 2023 10:00
Copy link
Member

@julien-nc julien-nc left a comment

Choose a reason for hiding this comment

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

🚀

@julien-nc
Copy link
Member

julien-nc commented Aug 9, 2023

Psalm issue because missing core/ResponseDefinitions.php. Even if we add this file, the OCA\Core namespace is not valid in stable27.
@provokateurin Any clue?

@provokateurin
Copy link
Member

The annotations are not available in stable27. The are only on master

@marcelklehr
Copy link
Member Author

The annotations are not available in stable27.

Ah, right. then we need to remove them

@marcelklehr marcelklehr added this to the Nextcloud 27.1.0 milestone Aug 9, 2023
@marcelklehr
Copy link
Member Author

🎉

@juliusknorr juliusknorr added enhancement 3. to review Waiting for reviews labels Aug 9, 2023
version.php Outdated Show resolved Hide resolved
Copy link
Member

@juliusknorr juliusknorr left a comment

Choose a reason for hiding this comment

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

Looks good from my side, just left a comment on the version bump to clarify before merge

julien-nc and others added 9 commits August 9, 2023 14:58
…tifier

Signed-off-by: Julien Veyssier <[email protected]>
(cherry picked from commit 9986e02)
Signed-off-by: Julien Veyssier <[email protected]>
add a textprocessing_tasks index
convert anotations to method attributes
refactor TP manager
add mapper methods

Signed-off-by: Julien Veyssier <[email protected]>

(cherry picked from commit 41b19cf)
Signed-off-by: Julien Veyssier <[email protected]>
Signed-off-by: Julien Veyssier <[email protected]>
(cherry picked from commit 05a6a79)
Signed-off-by: Julien Veyssier <[email protected]>
Signed-off-by: Julius Härtl <[email protected]>
(cherry picked from commit fca1c30)
Signed-off-by: Julien Veyssier <[email protected]>
Signed-off-by: Julien Veyssier <[email protected]>
(cherry picked from commit f154fe7)
Signed-off-by: Julien Veyssier <[email protected]>
Signed-off-by: Julien Veyssier <[email protected]>
Signed-off-by: Julien Veyssier <[email protected]>
Signed-off-by: Julien Veyssier <[email protected]>
Signed-off-by: Julien Veyssier <[email protected]>
@julien-nc
Copy link
Member

Removed the 2 commits changing version.php, added one bumping the last version digit. Put the "compile assets" commit on top.

@marcelklehr marcelklehr merged commit b2589c8 into stable27 Aug 9, 2023
38 checks passed
@marcelklehr marcelklehr deleted the backport/39680/stable27 branch August 9, 2023 13:47
@skjnldsv skjnldsv mentioned this pull request Aug 16, 2023
7 tasks
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 enhancement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants