icesat2py / icepyx

Python tools for obtaining and working with ICESat-2 data
https://icepyx.readthedocs.io/en/latest/
BSD 3-Clause "New" or "Revised" License
219 stars 107 forks source link

Workflow to approve integration tests from forks #588

Closed mfisher87 closed 1 month ago

mfisher87 commented 2 months ago

Follow up on #580.

[!IMPORTANT]

Preventing Pwn Requests is important.

Secrets exfiltration is a risk if we try to allow integration tests that use secrets to run on PRs from forks.

I took a stab and ran in to a wall. Taking a pause on this to focus on other high priorities. I may be entirely misguided in my approach. Any expertise would be appreciated :)

github-actions[bot] commented 2 months ago

Binder :point_left: Launch a binder notebook on this branch for commit 2f9a4aa5ee0558f2129c751aab0f17afb509d493

I will automatically update this comment whenever this PR is modified

mfisher87 commented 1 month ago

Superceded by #617