publiclab / webjack

a JavaScript library that uses an audio software modem to communicate with an Arduino via a headphone jack
https://webjack.io
GNU General Public License v3.0
136 stars 26 forks source link

try migrating to GitHub Actions from Travis for CI #189

Closed aliciapaz closed 3 years ago

aliciapaz commented 3 years ago

Based on similar work on: https://github.com/publiclab/spectral-workbench.js/pull/207 and https://github.com/publiclab/image-sequencer/pull/1783

Let's see how this goes!

welcome[bot] commented 3 years ago

Thanks for opening this pull request! Dangerbot will test out your code and reply in a bit with some pointers and requests. There may be some errors, but don't worry! We're here to help! 👍🎉😄

aliciapaz commented 3 years ago

For some reason, when I open a PR from my feature branch, the workflow is not triggered. However, if I open a PR on my own forked repository, the GH action works: https://github.com/aliciapaz/webjack/pull/1 @jywarren what do you think might be wrong? On the other hand, this made me realize I can test the workflows in my forked repository before I open the PR in the base repo :)

jywarren commented 3 years ago

Aha! Yes I believe this is a security limitation. I believe we can get you credentials to open on publiclab's as well!

On Mon, Aug 2, 2021, 6:39 PM Alicia Paz @.***> wrote:

For some reason, when I open a PR from my feature branch, the workflow is not triggered. However, if I open a PR on my own forked repository, the GH action works: aliciapaz#1 https://github.com/aliciapaz/webjack/pull/1 @jywarren https://github.com/jywarren what do you think might be wrong? On the other hand, this made me realize I can test the workflows in my forked repository before I open the PR in the base repo :)

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/publiclab/webjack/pull/189#issuecomment-891380362, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAAF6J7BQVBDRGOQH6NTV5LT24NDRANCNFSM5BI54NWA . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&utm_campaign=notification-email .

jywarren commented 3 years ago

Hi Alicia - just wondering if we could sync up in Gitter or another channel? Thanks!

I checked permissions and i know there's something about it that's a bit hard to remember -- if I make you a contributor on this repo, you should be able to open it from a branch of publiclab/webjack, not aliciapaz/webjack, and i think that will help? But i'm not 100% sure.

This looks all good, for starters:

image

jywarren commented 3 years ago

This document is strangely unhelpful: https://docs.github.com/en/organizations/managing-organization-settings/disabling-or-limiting-github-actions-for-your-organization

jywarren commented 3 years ago

So glad to see you're digging into this!

jywarren commented 3 years ago

Moving to https://github.com/publiclab/webjack/pull/191 🎉