Skip to content

Fix ruby lsp #510

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 5 commits into
base: main
Choose a base branch
from
Open

Fix ruby lsp #510

wants to merge 5 commits into from

Conversation

danhalson
Copy link
Contributor

Status

Ready for review

What's changed?

  • Fix ruby-lsp inside dev-container, previously wasn't working as asdf (or any verison manager) wasn't on the image.
  • Defines zsh as default terminal (previous cmd was deprecated apparently)
  • Puts the bundle on a volume for the dev-container
  • Make the smee tunnel an overwritable value

Steps to perform after deploying to production

  • Rebuild your dev containers with Dev Containers: Rebuild Container cmd in vscode

@danhalson danhalson self-assigned this Feb 20, 2025
@cla-bot cla-bot bot added the cla-signed label Feb 20, 2025
@raspberrypiherokubot raspberrypiherokubot temporarily deployed to editor-api-p-fix-ruby-l-xs7oni February 20, 2025 15:37 Inactive
@danhalson danhalson temporarily deployed to editor-api-p-fix-ruby-l-xs7oni February 21, 2025 08:53 Inactive
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@danhalson danhalson

Labels
cla-signed
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@danhalson @raspberrypiherokubot