Closed manics closed 4 years ago
Thanks for submitting your first pull request! You are awesome! :hugs:
If you haven't done so already, check out Jupyter's Code of Conduct. Also, please make sure you followed the pull request template, as this will help us review your contribution more quickly.
You can meet the other Jovyans by joining our Discourse forum. There is also a intro thread there where you can stop by and say Hi! :wave:
Welcome to the Jupyter community! :tada:
See an example on https://github.com/manics/idr0083-sars-cov2-notebook/pull/5
@hamelsmu this uses pull_request_target
: https://github.blog/2020-08-03-github-actions-improvements-for-fork-and-pull-request-workflows/#improvements-for-public-repository-forks
Yep can confirm this works. This feature came out yesterday. Cheers
gah this is cool!
@choldgraf can we merge this?
I don't know the infra on this one super well so am probably not best to say if (technically-speaking) it's good to merge, but I trust y'alls judgment 👍
@choldgraf you are going to have to push the button, I don't have the permissions
🤦♂️ my bad!
Congrats on your first merged pull request in this project! :tada: Thank you for contributing, we are very proud of you! :heart:
binder-badge.yml will now work with pull requests from forks, using the new
pull_request_target
. the action is also changed to use the commit sha instead of the branch name- since it's automatically run after every push to a PR this probably makes more sense than repeatedly posting a badge with the same branch name.