-
-
Notifications
You must be signed in to change notification settings - Fork 926
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
Temporarily host REM on fly to fix the docs #2893
Conversation
Can you also do a search and replace on |
Sorry missed this, can you link me to the PR where Pat did that and I'll do my best in a separate PR. |
@JAForbes I'll do it. I have some spare time. |
And fixed with a manual push: 487780f Here's what I did, for future reference:
|
Description
Fixes REM examples in the docs.
The documentation currently has a dead link as REM is no longer hosted on heroku. I think we should probably rewrite the example to not rely on an external service, but in the meantime I've hosted Leo's REM service on fly.io.
Motivation and Context
It's not good having broken documentation for the central example of the framework
How Has This Been Tested?
I updated the flems to use the new service and it worked. You can try it out here
Types of changes
Checklist:
docs/changelog.md