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

Italicize titles #73

Open
oliviagallucci opened this issue Jun 28, 2022 · 1 comment
Open

Italicize titles #73

oliviagallucci opened this issue Jun 28, 2022 · 1 comment

Comments

@oliviagallucci
Copy link
Contributor

There are a lot of "2021 CWE Tip 25" and "2019 CWE Top 25". It gets hard to follow when they are not italicized because the sentences are so long.

All of the CWE and OWASP lists should be in italics IMO.

@david-a-wheeler
Copy link
Contributor

That would make some sense. That change would happen many times, so we probably need to do a find & replace. I'll rope in Flavia before we start doing that.

david-a-wheeler added a commit that referenced this issue Jun 29, 2022
Italicize 20.. CWE Top 25 and 20.. OWASP Top 10 titles.
Note that the CWE Top 25 title sometimes includes
"Most Dangerous Software Errors". In one case it was already
italicized, so we revert the doubled asterisks, and
we also fix a missing space after an "and" before italicizing it.

This is the result of:

~~~~sh
cat secure_software_development_fundamentals.md | \
  sed -E 's/(20.. CWE Top 25( Most Dangerous Software Errors)?)/\*\1\*/g' | \
  sed -E \
    's/\*(\*20.. CWE Top 25( Most Dangerous Software Errors)?\*)\*/\1/g' | \
  sed -E 's/(20.. OWASP Top 10)/\*\1\*/g' | \
  sed -E 's/ and\*/ and \*/g' > ,1
  mv ,1 secure_software_development_fundamentals.md
~~~~

Signed-off-by: David A. Wheeler <dwheeler@dwheeler.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants