Allow passing in GH token #31
Conversation
|
Shouldn't the CI script then also pass in the auth token? |
I have a diff here: https://github.com/axmmisaka/action-setup-lf/commit/7f60b1ea9a05998ef2976f8c79ebecc3843c70b2 |
|
How did you even figure out that this was a problem? 🤣 Note that this fix will only work in CI, not in actual use of the installer. If we don't want to be rate limited, we should probably just serve this differently. Could we just serve it via Pages? I imagine that wouldn't be rate limited as aggressively. |
I tweaked the script to have https://github.com/mxschmitt/action-tmate this is the ugliest form of debugging but works nicely
One use case I could think of is if people want to install LF onto a cluster of machines with this script (they shouldn't), they can supply their own token.
To be honest I'm not 100% sure... |
GH API might rate-limit us which is a concern when running CI that fires a bunch of curl requests unless we provide a GH token. This PR allows us to do so.