Skip to content

Commit

Permalink
Merge pull request #923 from Shopify/uk-add-ruby-lsp
Browse files Browse the repository at this point in the history
Add `ruby-lsp` dependency
  • Loading branch information
paracycle committed May 4, 2022
2 parents e2dfe3e + bd59598 commit 409e08d
Show file tree
Hide file tree
Showing 6 changed files with 29 additions and 20 deletions.
19 changes: 10 additions & 9 deletions .vscode/extensions.json
@@ -1,13 +1,14 @@
{
// See https://go.microsoft.com/fwlink/?LinkId=827846 to learn about workspace recommendations.
// Extension identifier format: ${publisher}.${name}. Example: vscode.csharp
// See https://go.microsoft.com/fwlink/?LinkId=827846 to learn about workspace recommendations.
// Extension identifier format: ${publisher}.${name}. Example: vscode.csharp

// List of extensions which should be recommended for users of this workspace.
"recommendations": [
"Shopify.ruby-extensions-pack"
],
// List of extensions recommended by VS Code that should not be recommended for users of this workspace.
"unwantedRecommendations": [
// List of extensions which should be recommended for users of this workspace.
"recommendations": [
"Shopify.ruby-extensions-pack",
"Shopify.ruby-lsp"
],
// List of extensions recommended by VS Code that should not be recommended for users of this workspace.
"unwantedRecommendations": [

]
]
}
2 changes: 1 addition & 1 deletion .vscode/settings.json
Expand Up @@ -10,7 +10,7 @@
"byesig.fold": false,
"byesig.showIcon": false,
"[ruby]": {
"editor.defaultFormatter": "Shopify.rubocop-lsp",
"editor.defaultFormatter": "Shopify.ruby-lsp",
"editor.formatOnSave": true
}
}
2 changes: 1 addition & 1 deletion Gemfile
Expand Up @@ -12,6 +12,7 @@ gem("rubocop-shopify", require: false)
gem("rubocop-sorbet", ">= 0.4.1")
gem("rubocop-rspec", require: false)
gem("sorbet")
gem("ruby-lsp", require: false)

group(:deployment, :development) do
gem("rake")
Expand Down Expand Up @@ -41,7 +42,6 @@ group(:development, :test) do
gem("aasm", require: false)
gem("bcrypt", require: false)
gem("xpath", require: false)
gem("rubocop-lsp", require: false)

# net-smtp was removed from default gems in Ruby 3.1, but is used by the `mail` gem.
# So we need to add it as a dependency until `mail` is fixed:
Expand Down
10 changes: 6 additions & 4 deletions Gemfile.lock
Expand Up @@ -256,15 +256,16 @@ GEM
unicode-display_width (>= 1.4.0, < 3.0)
rubocop-ast (1.17.0)
parser (>= 3.1.1.0)
rubocop-lsp (0.1.6)
language_server-protocol (>= 3.16)
rubocop (>= 1.0)
rubocop-rspec (2.10.0)
rubocop (~> 1.19)
rubocop-shopify (2.5.0)
rubocop (~> 1.25)
rubocop-sorbet (0.6.8)
rubocop (>= 0.90.0)
ruby-lsp (0.0.2)
language_server-protocol
rubocop (>= 1.0)
syntax_tree (>= 2.3)
ruby-progressbar (1.11.0)
shopify-money (0.16.0)
sidekiq (6.4.2)
Expand Down Expand Up @@ -297,6 +298,7 @@ GEM
sqlite3 (1.4.2)
state_machines (0.5.0)
strscan (3.0.1)
syntax_tree (2.3.1)
thor (1.2.1)
timeout (0.2.0)
tzinfo (2.0.4)
Expand Down Expand Up @@ -339,10 +341,10 @@ DEPENDENCIES
pry-byebug
rails
rake
rubocop-lsp
rubocop-rspec
rubocop-shopify
rubocop-sorbet (>= 0.4.1)
ruby-lsp
shopify-money
sidekiq
smart_properties
Expand Down

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

8 changes: 8 additions & 0 deletions sorbet/rbi/gems/syntax_tree@2.3.1.rbi

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

0 comments on commit 409e08d

Please sign in to comment.