-
-
Notifications
You must be signed in to change notification settings - Fork 83
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
Improve incompatible version errors on boot #389
Conversation
1a2e4f1
to
7671a34
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Have you tried running this under incompatible erlang versions? I don't think it'll even boot.
You're right, it doesn't. That check was there previously, however, so I figured there might be some circumstances where it does boot and then fails later. |
bfbaafd
to
c07c792
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
My main question is if this will even work for erlang version mismatches. I think it'll only work if we were compiled on an older version and are running on a newer version.
c07c792
to
623683d
Compare
@scohen Updated this based on your feedback.
I think you're right -- |
Yes. |
This commit makes a few improvements to the errors shown when Lexical boots with incompatible Elixir/Erlang versions: * Consistent formatting for all errors * Prevent an exception that would occur if using an incompatible major version (e.g. Elixir 1.12) * If both Elixir and Erlang are incompatible, show both errors * Show full range of compatible versions instead of just the range for the current major version
7771389
to
baf91e2
Compare
This commit makes a few improvements to the errors shown when Lexical boots with incompatible Elixir/Erlang versions: * Consistent formatting for all errors * Prevent an exception that would occur if using an incompatible major version (e.g. Elixir 1.12) * If both Elixir and Erlang are incompatible, show both errors * Show full range of compatible versions instead of just the range for the current major version
This commit makes a few improvements to the errors shown when Lexical boots with incompatible Elixir/Erlang versions:
Example before:
Example after: