Make congratsbot not run in forks #1145
Conversation
|
|
This pull request is being automatically deployed with Vercel (learn more). astro-www – ./www🔍 Inspect: https://vercel.com/pikapkg/astro-www/Exwa1hyF5Y8Ga2Wn7uZLQhq8Pk6Y astro-docs – ./docs🔍 Inspect: https://vercel.com/pikapkg/astro-docs/HfbUYuJN91Uxxm5WMswKUkBGDE2F |
|
LGTM! Only one way to test :) |
* wip * Add support for `client:only` hydrator (#935) * Adding support for client:only hydration * Adding documentation for client:only * Adding changeset * Updating the test to use a browser-only API * Adding a browser-specific import script, this reproduces the issue where client:only imports must be removed * typo fix * removing mispelled test component * WIP: delaying inclusion of component imports until the hydration method is known * WIP: tweaking the test to use window instead of document * When only one renderer is included, use that for client:only hydration * temporary test script snuck into the last commit * WIP: adding check for a client:only renderer hint * refactor: Remove client:only components instead of delaying all component import statements * Updating the changeset and docs for the renderer hint * refactor: pull client:only render matching out to it's own function * Updating renderer hinting to match full name, with shorthand for internal renderers Co-authored-by: Tony Sullivan <[email protected]> * [ci] yarn format * Update CONTRIBUTING.md (#1131) * [ci] yarn format * docs: fix select language in Safari (#1127) (#1128) * docs: fix select language in Safari (#1127) * docs: fix select language top position * docs: fix select language position * Make congratsbot not run in forks (#1145) * add back dark-mode aware favicons * make example favicons prefer non-dark mode * Version Packages (next) (#1129) Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> * code review comments Co-authored-by: Tony Sullivan <[email protected]> Co-authored-by: Tony Sullivan <[email protected]> Co-authored-by: matthewp <[email protected]> Co-authored-by: FredKSchott <[email protected]> Co-authored-by: Oleg <[email protected]> Co-authored-by: Marcus Otterström <[email protected]> Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Changes
Makes this workflow not run in forks, since it sends emails to me when it fails:
Testing
I have not, and I have no idea how to test 😅
I just saw this solution in an issue on https://github.com/actions/runner/, and also that most workflows use this in quarkus.io.
Definitely test somehow before merging! (or don't, I mean, the only effect it has is that I don't get a congratulation when the PR is merged if the code is wrong)
Docs
No