Skip to content

Conversation

JasonEtco
Copy link
Contributor

Why:

Closes #97

What's being changed:

When I modified this workflow's message, I didn't properly escape the ` characters in the code block 🙈

Check off the following:

@JasonEtco JasonEtco requested a review from a team as a code owner October 7, 2020 19:21
@welcome
Copy link

welcome bot commented Oct 7, 2020

Thanks for opening this pull request! A GitHub docs team member should be by to give feedback soon. In the meantime, please check out the contributing guidelines.

Copy link
Contributor

@zeke zeke left a comment

Choose a reason for hiding this comment

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

👍🏼

@JasonEtco JasonEtco merged commit ee6f5ae into main Oct 7, 2020
@JasonEtco JasonEtco deleted the fix-merged-notif-action branch October 7, 2020 20:01
@github-actions
Copy link
Contributor

github-actions bot commented Oct 7, 2020

Thanks very much for contributing! Your pull request has been merged 🎉 You should see your changes appear on the site in approximately 24 hours. To add your ✨ contribution to the README.md, create a new comment in this PR with:

@all-contributors please add @JasonEtco for docs

If you want to, you can use the emoji key to replace docs with a different contribution type.

@allcontributors
Copy link
Contributor

@github-actions[bot]

I could not determine your intention.

Basic usage: @all-contributors please add @Someone for code, doc and infra

For other usages see the documentation

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.

"merged notification" workflow is being skipped
2 participants