Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

String values in README.md extended example are now correct #196

Conversation

karolswdev
Copy link
Contributor

Hi @stefanzweifel - great work on your GitHub Action. While playing with it, I realized that your example doesn't contain quotes to populate a string value for your action. This PR fixes that. Gruesse.

Copy link
Owner

@stefanzweifel stefanzweifel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for your PR!
I'v added a comment regarding the changes to file_pattern.

As far as I know, one doesn't have to us quotes for strings in YAML. Except when one has special charcaters like : or # in them.

I personally like the "quote-less" style better and would like to keep it that way. Or did you run into issues with the current examples in the README? Any links where I could learn more about the issue?

README.md Outdated Show resolved Hide resolved
@stefanzweifel stefanzweifel merged commit 03246c1 into stefanzweifel:master Sep 17, 2022
# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants