Skip to content

Removes unnecessary newlines from rich text as JSON delivery API output #19391

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.

Sign up for GitHub

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

Merged
AndyButland merged 2 commits into from
May 23, 2025

Fix case from PR feedback.

14ba4d8
Select commit
Loading
Failed to load commit list.
Merged

Removes unnecessary newlines from rich text as JSON delivery API output #19391

Fix case from PR feedback.
14ba4d8
Select commit
Loading
Failed to load commit list.
CodeScene Delta Analysis / CodeScene Cloud Delta Analysis (v13/dev) succeeded May 23, 2025 in 41s

CodeScene PR Check

Quality Gate Passed

Gates Passed
4 Quality Gates Passed

See analysis details in CodeScene

Quality Gate Profile: Clean Code Collective
Want more control? Customize Code Health rules or catch issues early with our IDE extension and CLI tool.

Details

✅ Improving Code Health:

  • String Heavy Function Arguments RichTextParserTests.cs

Annotations

Check notice on line 1 in tests/Umbraco.Tests.UnitTests/Umbraco.Core/DeliveryApi/RichTextParserTests.cs

See this annotation in the file changed.

@codescene-delta-analysis codescene-delta-analysis / CodeScene Cloud Delta Analysis (v13/dev)

✅ Getting better: String Heavy Function Arguments 

The ratio of strings in function arguments decreases from 42.86% to 42.11%, threshold = 39.0%. The functions in this file have a high ratio of strings as arguments. Avoid adding more.
View more details on CodeScene Delta Analysis