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 comment on issues marked 'good first issue' #25735
Conversation
f6708ca
to
938e01b
Compare
@melissawm friendly reminder to have a look and possibly suggest a better wording. (I think we will have to try it to see how it works, but this is a reasonably well used action.) |
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 I missed this! Here's a suggestion for the wording - I also added a direct link to the "get connected" section.
Co-authored-by: Melissa Weber Mendonça <melissawm@gmail.com>
pre-commit doesn't like it; some trailing whitespace slipped in. |
Fixed that. Make sure to squeeze when merging. |
It seems to work: #25942 (although the link text makes the text in the comment line break in an ugly way because of the line length limitation). |
* Add comment on issues marked 'good first issue' * Apply suggestions Co-authored-by: Melissa Weber Mendonça <melissawm@gmail.com> * Remove trailing white space --------- Co-authored-by: Melissa Weber Mendonça <melissawm@gmail.com>
* Add comment on issues marked 'good first issue' * Apply suggestions Co-authored-by: Melissa Weber Mendonça <melissawm@gmail.com> * Remove trailing white space --------- Co-authored-by: Melissa Weber Mendonça <melissawm@gmail.com>
PR Summary
Who knows if this works? @melissawm feel free to update the text.
PR Checklist
Documentation and Tests
pytest
passes)Release Notes
.. versionadded::
directive in the docstring and documented indoc/users/next_whats_new/
.. versionchanged::
directive in the docstring and documented indoc/api/next_api_changes/
next_whats_new/README.rst
ornext_api_changes/README.rst