Closed simonhamp closed 9 months ago
@hrsa can you check this? looks dope.
@nunomaduro it is! I'd love to but I have no access to my laptop until Sunday evening, I can take a look soon 👍
Made a few improvements to this just to make it a bit more reliable. It now:
gh
CLI to create the PR (the action that it used has been archived)stats_{Y-m-d}
so that if you aren't able to approve the PRs each week, you won't get failed runs (any unmerged, old PRs from this process can likely just be closed and the branches deleted)@hrsa @nunomaduro anything I can do to help get this one over the line?
@simonhamp sorry for the delay, that's a great contribution ! 💪 @nunomaduro i've tested it out and it works well. but before merging - either you would have to add the API key to the secrets, or i would need admin access to do that :)
You are now admin.
No apologies necessary @hrsa ! Thanks for taking the time to review and merge 🎉
@nunomaduro we also need to modify the permissions for the workflow, since it creates PRs: https://github.com/php-youtubers/directory/settings/actions The options are greyed out for me, i guess that only you can modify that
@nunomaduro still need you here :)
@nunomaduro we also need to modify the permissions for the workflow, since it creates PRs: https://github.com/php-youtubers/directory/settings/actions The options are greyed out for me, i guess that only you can modify that
This simply adds a GitHub Action that should update the stats every Monday.
It requires you to set an
YOUTUBE_API_KEY
secret, which you should be able to do hereWhen the cron runs it will PR the updates, so it should be a simple case of merging the PR to keep the list up to date