Remove superfluous HTML span tags from SECURITY.md

[TobiasBg]

Core's SECURITY.md file is written in Markdown, and while that allows for using inline HTML code, there is no reason (at least I couldn't find one documented) to wrap all link texts in extra HTML <span> tags.

The PR ​https://github.com/WordPress/wordpress-develop/pull/3714 removes these extra tags to make the file more readable and maintainable.

Core's SECURITY.md file is written in Markdown, and while that allows for using inline HTML code, there is no reason (at least I couldn't find one documented) to wrap all link texts in extra HTML <span> tags.

This PR removes these extra tags to make the file more readable and maintainable.

Trac ticket: https://core.trac.wordpress.org/ticket/57243

[peterwilsoncc]

The linked pull request looks good to me as of commit hash ​88c6fcd27b.

While committing changes to the security policy recently, I noticed that npm run grunt patch had problems with the upper case file name. The following command will work but requires the GitHub CLI:

gh pr diff 3714 --repo WordPress/wordpress-develop | patch -p1

Yes, I also couldn't spot visual or functional differences.
In fact, there are already at least 3 Markdown links in the file that don't have these <span> tags.

My guess is that maybe a Markdown editor program added these HTML tags, when the documented was first written, or that these come from some copy/paste?

Pinging @whyisjake as the original committer of the file, in case he remembers details or has objections here.

... or that these come from some copy/paste?

My money is on this too, especially if it was originally written in a Google Doc/Word or similar. They're 🤬 for adding superfluous spans via the clipboard.

[audrasjb]

Self assigning for commit.

[audrasjb]

In 54934:

Security: Remove useless span tags from SECURITY.md.

Props TobiasBg, peterwilsoncc.
Fixes #57243.

Committed in https://core.trac.wordpress.org/changeset/54934