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

README: make some passages more explicit #165

Merged
merged 4 commits into from
Nov 1, 2021

Conversation

waldyrious
Copy link
Contributor

Note: The changes in the comments of the example YAML file are both to improve clarity, and to avoid the need to scroll the sample text horizontally to read the comments.

Follow-up of #148.

Repository owner deleted a comment from github-actions bot Nov 1, 2021
Repository owner deleted a comment from github-actions bot Nov 1, 2021
@dangoslen
Copy link
Owner

Sorry for the actions not working properly. I have some fixing to do there I guess.

These changes look good to me. I'm not one to argue with clarity since my opinion is always the more biased one :)

I'll give a quick glance and merge if I don't see anything worth changing

@dangoslen dangoslen merged commit 238e10c into dangoslen:master Nov 1, 2021
@waldyrious waldyrious deleted the patch-1 branch November 1, 2021 23:40
@waldyrious
Copy link
Contributor Author

Awesome! Thanks for the quick review and merge :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants