Skip to content
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

r: Bump version of rsession-proxy #1310

Merged
merged 2 commits into from
Oct 12, 2023
Merged

Conversation

yuvipanda
Copy link
Collaborator

@yuvipanda yuvipanda commented Oct 12, 2023

If memory serves me right, the specific version semantics
were here because at some point we were installing different
versions of rsession-proxy for different versions of R. We
don't do that anymore, so we should just unpin it and let the
pip solver install the appropriate version.

Hopefully fixes https://discourse.jupyter.org/t/r-conda-example-repo-isnt-working-any-more/21739/3
fomightez added a commit to fomightez/rcondatestOct that referenced this pull request Oct 12, 2023
@manics
Copy link
Member

manics commented Oct 12, 2023

I think we're pinning everything else that's installed by default in r2d. Is there anyway we can move this to a requirements.txt file that can be bumped by dependabot (or the freeze.py script, though dependabot is easier)?

We can move to a proper pinning setup later
@yuvipanda
Copy link
Collaborator Author

@manics I've instead just bumped the version number to latest to minimize the amount of changes we need to (hopefully?) fix the current issue. Given I'm trying to focus on the frontend on binderhub, I don't think I currently have time to move things to a requirements.txt. I hope this is acceptable instead.

@yuvipanda yuvipanda changed the title r: Don't strictly pin version of rsession-proxy r: Bump version of rsession-proxy Oct 12, 2023
Copy link
Member

@manics manics left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! I'll open a separate issue about auto-bumping these r packages

@yuvipanda yuvipanda merged commit 8c32db9 into jupyterhub:main Oct 12, 2023
18 checks passed
@yuvipanda
Copy link
Collaborator Author

Thanks, @manics!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants