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

Use yaml for package data instead of json #569

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

uncenter
Copy link

@uncenter uncenter commented Sep 27, 2023

This is a continuation of sorts of #567 - I noticed a lot of issues in the md-variables.json file probably arose due to how difficult it was to edit arrays of strings! YAML isn't everyone's favorite, but I think it really shines here with the multiline strings. This PR converts the md-variables.json file to YAML, including my typo fixes from #567, and updates the scripts/generate-readmes.js script accordingly to parse the YAML.

scripts/generate-readmes.js Outdated Show resolved Hide resolved
@uncenter uncenter marked this pull request as ready for review September 27, 2023 23:23
@uncenter
Copy link
Author

uncenter commented Oct 8, 2023

You can see the changes made in this PR by cloning this branch locally and running yarn generate-readmes. The only changes are intentional punctuation changes - there are no changes due to changing the data format.

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.

1 participant