Skip to content

Add YAML normalization#1110

Merged
aduth merged 3 commits intomainfrom
aduth-normalize-yaml
Jun 30, 2023
Merged

Add YAML normalization#1110
aduth merged 3 commits intomainfrom
aduth-normalize-yaml

Conversation

@aduth
Copy link
Copy Markdown
Contributor

@aduth aduth commented Jun 30, 2023

🛠 Summary of changes

Adds an enforces YAML normalization, using the @18f/identity-normalize-yaml package published from identity-idp in 18F/identity-idp#8654.

Why?

  • Improved typography using punctuation formatters
    • Also resolves some existing inconsistency where we previously used smart quotes in some places, either due to Jekyll's built-in smart quotes handling or where content was copied from some other source
  • Consistency of content formatting in YAML files
    • Improved readability by enforced key alphabetization

📜 Testing Plan

make lint_yaml

Ensure no regressions in displayed content in preview site.

@aduth aduth merged commit f7d5a79 into main Jun 30, 2023
@aduth aduth deleted the aduth-normalize-yaml branch June 30, 2023 17:08
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

Successfully merging this pull request may close these issues.

2 participants