Skip to content

Conversation

axelson
Copy link
Member

@axelson axelson commented May 21, 2020

Vendor the Jason library so that the version of Jason used by ElixirLS does not conflict with the version used by the end-users code. Right now one will win out over the other which can lead to issues. Luckily since Jason has been pretty stable there haven't been too many issues up to this point.

Pulls the vendored version of Jason from https://github.com/elixir-lsp/jason

Fixes #253

Vendor the Jason library so that the version of Jason used by ElixirLS does not
conflict with the version used by the end-users code. Right now one will win out
over the other which can lead to issues. Luckily since Jason has been pretty
stable there haven't been too many issues up to this point.

Fixes #253
mix.lock Outdated
"erlex": {:hex, :erlex, "0.2.6", "c7987d15e899c7a2f34f5420d2a2ea0d659682c06ac607572df55a43753aa12e", [:mix], [], "hexpm", "2ed2e25711feb44d52b17d2780eabf998452f6efda104877a3881c2f8c0c0c75"},
"forms": {:hex, :forms, "0.0.1", "45f3b10b6f859f95f2c2c1a1de244d63855296d55ed8e93eb0dd116b3e86c4a6", [:rebar3], [], "hexpm", "530f63ed8ed5a171f744fc75bd69cb2e36496899d19dbef48101b4636b795868"},
"getopt": {:hex, :getopt, "1.0.1", "c73a9fa687b217f2ff79f68a3b637711bb1936e712b521d8ce466b29cbf7808a", [:rebar3], [], "hexpm", "53e1ab83b9ceb65c9672d3e7a35b8092e9bdc9b3ee80721471a161c10c59959c"},
"jason": {:hex, :jason, "1.2.1", "12b22825e22f468c02eb3e4b9985f3d0cb8dc40b9bd704730efa11abd2708c44", [:mix], [{:decimal, "~> 1.0", [hex: :decimal, repo: "hexpm", optional: true]}], "hexpm", "b659b8571deedf60f79c5a608e15414085fa141344e2716fbd6988a084b5f993"},
Copy link
Collaborator

Choose a reason for hiding this comment

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

Is the original jason still required by some other dependancy?

Choose a reason for hiding this comment

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

The little forgotten mix deps.unlock --unused can remove the stale info from the lock file.

Copy link
Member Author

Choose a reason for hiding this comment

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

Ah, good catch. I always forget to run mix deps.unlock --unused

@axelson axelson merged commit 2e28933 into elixir-lsp:master May 22, 2020
@axelson axelson deleted the vendor-jason branch May 22, 2020 16:40
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.

Crash: protocol Jason.Encoder not implemented for ElixirLS.LanguageServer.Protocol.DocumentSymbol
3 participants