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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

[stable23] Remove iconv from dependencies and tests #30351

Merged
merged 1 commit into from
Jan 6, 2022

Conversation

MichaIng
Copy link
Member

@MichaIng MichaIng commented Dec 20, 2021

Manual backport of: #29958
As of: #29965

I removed iconv from some additional tests which were missed before, assured to catch all cases via grep -ir now 馃槈. Done for master here: #30352

Requires a backport of the documentation change: nextcloud/documentation#7860

@MichaIng MichaIng added the 3. to review Waiting for reviews label Dec 20, 2021
@MichaIng MichaIng added this to the Nextcloud 23.0.1 milestone Dec 20, 2021
@MichaIng MichaIng added the pending documentation This pull request needs an associated documentation update label Dec 20, 2021
@MichaIng
Copy link
Member Author

/backport to stable22

@MichaIng
Copy link
Member Author

/backport to stable21

@MichaIng
Copy link
Member Author

Related docs update: nextcloud/documentation#7864

which is not used anymore since: #29470

Signed-off-by: MichaIng <micha@dietpi.com>
@blizzz blizzz merged commit 994b32f into stable23 Jan 6, 2022
@blizzz blizzz deleted the backport/29958/stable23 branch January 6, 2022 22:37
@backportbot-nextcloud
Copy link

The backport to stable22 failed. Please do this backport manually.

@backportbot-nextcloud
Copy link

The backport to stable21 failed. Please do this backport manually.

@MichaIng
Copy link
Member Author

MichaIng commented Jan 6, 2022

Manual backport to stable22: #30515

@skjnldsv skjnldsv mentioned this pull request Jan 7, 2022
6 tasks
This was referenced Jan 20, 2022
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 pending documentation This pull request needs an associated documentation update
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants