FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Avoid unnecessarily early escape of Site Health check labels by mukeshpanchal27 · Pull Request #332 · WordPress/performance · GitHub

Avoid unnecessarily early escape of Site Health check labels - #332

Merged
felixarntz merged 3 commits into
WordPress:release/1.1.0from
mukeshpanchal27:fix/correct-escaping
May 10, 2022
Merged

Avoid unnecessarily early escape of Site Health check labels#332
felixarntz merged 3 commits into
WordPress:release/1.1.0from
mukeshpanchal27:fix/correct-escaping

Conversation

mukeshpanchal27 commented May 10, 2022
edited by felixarntz
Loading

Copy link
Copy Markdown
Member

As suggested by @felixarntz in the #329 ticket.

Checklist

  • PR has either [Focus] or Infrastructure label.
  • PR has a [Type] label.
  • PR has a milestone or the no milestone label.

felixarntz changed the title Correct escaping function Avoid unnecessarily early escape of Site Health check labels May 10, 2022

felixarntz left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Great, thank you @mukeshpanchal27!

felixarntz requested a review from tillkruss May 10, 2022 17:52
felixarntz added [Type] Enhancement A suggestion for improvement of an existing feature Infrastructure Issues for the overall performance plugin infrastructure labels May 10, 2022
felixarntz added this to the 1.1.0 milestone May 10, 2022
felixarntz changed the base branch from trunk to release/1.1.0 May 10, 2022 19:52

Copy link
Copy Markdown
Member

@mukeshpanchal27 I've updated the base branch to release/1.1.0, with the goal to get this into the upcoming release.

Can someone else please approve this by tomorrow? cc @adamsilverstein @manuelRod @kirtangajjar @mitogh @tillkruss 🙌

felixarntz merged commit 6e72e47 into WordPress:release/1.1.0 May 10, 2022
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Infrastructure Issues for the overall performance plugin infrastructure [Type] Enhancement A suggestion for improvement of an existing feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants


Back | FazBrowse Home | New Git URL