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

Add Rule to Format Yaml Arrays #336

Merged
merged 16 commits into from
Aug 13, 2022
Merged

Add Rule to Format Yaml Arrays #336

merged 16 commits into from
Aug 13, 2022

Conversation

pjkaufman
Copy link
Collaborator

@pjkaufman pjkaufman commented Aug 5, 2022

Fixes #172
Fixes #291
Fixes #182

Changes Made:

  • Added a rule to account for updating yaml array formats and forcing a regular string value to be a yaml array
  • Added a bunch of tests to handle some possible test cases
  • Added a couple of examples
  • Added a fix for a bug where if a url had [ in it, it was considered to be a regular markdown link which caused multiple < to be added at the start of the link when the rule for removing space around link text was active

@pjkaufman pjkaufman added the rule suggestion Suggestion to add or edit a rule label Aug 5, 2022
@pjkaufman pjkaufman self-assigned this Aug 5, 2022
@pjkaufman pjkaufman marked this pull request as draft August 5, 2022 21:01
@pjkaufman pjkaufman marked this pull request as ready for review August 5, 2022 23:17
@pjkaufman pjkaufman marked this pull request as draft August 6, 2022 00:19
@pjkaufman
Copy link
Collaborator Author

pjkaufman commented Aug 6, 2022

I need to fix that it is removing the quotation marks around single and double quotes in the yaml arrays when it adds them back.

@pjkaufman pjkaufman marked this pull request as ready for review August 13, 2022 15:57
@pjkaufman pjkaufman merged commit 2a9c973 into platers:master Aug 13, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
rule suggestion Suggestion to add or edit a rule
Projects
None yet
Development

Successfully merging this pull request may close these issues.

FR: Format YAML Arrays FR: Convert Aliases to Multiline Array FR: Format YAML 'tags' Key
1 participant