-
-
Notifications
You must be signed in to change notification settings - Fork 5.5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Restore or replace Heroku Review Apps #7924
Comments
One of the challenges I think we'd run into with the Fly review apps model is that it's predicated on having a GitHub Actions secret available, but those are disabled and unavailable on PRs coming from forks for security reasons |
https://blog.heroku.com/github-integration-update (some progress but still not resolved) |
Good find and good point about the fly review apps. I think in the short term re-enabling Heroku review apps when they enable github deploys again is probably the play. @SnoozeThis wait until May 26 2022 |
(https://snoozeth.is/X6ov_cwqaA8) I will wait until Thu, 26 May 2022 00:00:00 UTC and then add a comment. |
Resolved. |
partially done in but not really |
I know we've discussed already, but it continues to surprise me that the review apps for squint just continue to work while the ones for shields are all out of whack 🤷♂️ (e.g. badges/squint#111) |
Presumably we also had to re-connect the squint ones after the long heroku security outage. I wonder if we did anything differently? |
That's the thing, I didn't. Squint just started working |
Haunted. No other explanation. |
👻 |
Looks like our review apps are now failing with |
📋 Description
Until https://status.heroku.com/incidents/2413 happened we were using Heroku review apps to build a preview site for each PR. This was a big help for reviewing contributions and dependency bumps.
The latest update from Heroku is
The key phrase there for me is:
Investigate how we could re-enable review apps, either using Heroku via some other avenue or by migrating away from Heroku. One possible thing we could look at is https://github.com/superfly/fly-pr-review-apps
The text was updated successfully, but these errors were encountered: