Skip to content

Fix ruby lsp#510

Closed
danhalson wants to merge 5 commits intomainfrom
fix-ruby-lsp
Closed

Fix ruby lsp#510
danhalson wants to merge 5 commits intomainfrom
fix-ruby-lsp

Conversation

@danhalson
Copy link
Copy Markdown
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
@github-actions
Copy link
Copy Markdown

This PR is stale because it has been open for 30 days with no activity. Remove stale label or comment or this will be closed in 7 days.

@github-actions github-actions Bot added the Stale label Jan 29, 2026
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Feb 6, 2026

This PR was closed because it has been open for more than 30 days with no activity.

@github-actions github-actions Bot closed this Feb 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants