Skip to content

normalize-yaml: Enhance to collapse spacing#9980

Merged
aduth merged 4 commits intomainfrom
aduth-normalize-yaml-collapse-spaces
Jan 29, 2024
Merged

normalize-yaml: Enhance to collapse spacing#9980
aduth merged 4 commits intomainfrom
aduth-normalize-yaml-collapse-spaces

Conversation

@aduth
Copy link
Contributor

@aduth aduth commented Jan 26, 2024

🛠 Summary of changes

Adds new behavior to the YAML normalization package to collapse spacing between sentences in YAML content.

Related discussion: https://gsa-tts.slack.com/archives/CEUQ9FXNJ/p1706111249614589

📜 Testing Plan

Verify that make lint_yaml is produces a successful exit code.

  1. make lint_yaml
  2. echo $?
  3. Observe: 0

Verify that reverting included normalizations and running normalization applies the changes to collapse spaces

  1. git checkout main -- config/locales
  2. make normalize_yaml
  3. git add config/locales
  4. git status
  5. Observe "nothing to commit, working tree clean"

Verify that included specs pass:

  1. yarn mocha app/javascript/packages/normalize-yaml/index.spec.js

Loading
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.

3 participants