-
Notifications
You must be signed in to change notification settings - Fork 4.8k
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
repokitteh: Add welcome message for new contributors #13309
Conversation
cc @itayd |
a9b8780
to
0f7a98d
Compare
0f7a98d
to
221249a
Compare
298eccf
to
221249a
Compare
Signed-off-by: Ryan Northey <[email protected]>
ece29b7
to
5ad3a30
Compare
The output from the welcome message can be seen here #13306 (comment) |
Signed-off-by: Ryan Northey <[email protected]>
@itayd i think this one should be ready for review |
/retest |
Retrying Azure Pipelines, to retry CircleCI checks, use |
@mattklein123 it would be good to land this, and i would like to test it once it does |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Super cool, just one small comment.
/wait
@@ -0,0 +1,31 @@ | |||
|
|||
NEW_CONTRIBUTOR_MESSAGE = """ | |||
hi @%s, welcome and thankyou for your contribution. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
hi @%s, welcome and thankyou for your contribution. | |
Hi @%s, welcome and thank you for your contribution. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry, can you capitalize Hi?
/wait
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ah, sorry, on it...
Signed-off-by: Ryan Northey <[email protected]>
Signed-off-by: Ryan Northey <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
@mattklein123 this doesnt seem to have worked on the first pass. I have pinged @itayd to see if we can trace why not |
Signed-off-by: Ryan Northey [email protected]
Commit Message: repokitteh: Add welcome message for new contributors
Additional Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
[Optional Runtime guard:]
[Optional Fixes #Issue] Partial fix for #13232
[Optional Deprecated:]