-
Notifications
You must be signed in to change notification settings - Fork 15
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
chore: update rstudio version #179
Conversation
No luck with a quick PR. Rstudio does not start with this change. So we need to do some troubleshooting. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
as far as I can tell from running the resulting image locally, this should solve the problem reported in the discourse post.
Ah sorry, hadn't seen your comment on the issue. I just had a quick look why acceptance tests on this PR were failing and fixed it by adjusting the |
So does this work or not? |
not yet |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lets make sure it works :)
ok - I was confused by all the green and was tempted to push the merge button 😆 |
Yeah - I think in the light of recent events and our future plans for Rstudio sessions in renkulab.io, I'd suggest to close this PR altogether. |
I still have to test this to see if it works and fixes the issue reported here:
https://renku.discourse.group/t/rstudio-version-too-old/393