Skip to content
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

Add otp-25 support #988

Merged
merged 7 commits into from
Jan 3, 2023
Merged

Add otp-25 support #988

merged 7 commits into from
Jan 3, 2023

Conversation

zyuyou
Copy link
Contributor

@zyuyou zyuyou commented Aug 12, 2022

  1. Add otp-25 maybe_expr support. OTP 25 #970
  2. Fix "Setup SDK" suggests. "Setup SDK" button suggests to configure the JDK isntead of the Erlang SDK #977
  3. Fix rebar version check. can't check rebar version #903

@kvakvs
Copy link
Collaborator

kvakvs commented Nov 16, 2022

Code looks good
Also fixes one bug which i tried to fix myself, in an experimental branch (Setup SDK button
I'll give it a try soon

# Conflicts:
#	src/org/intellij/erlang/inspection/ErlangSdkNotificationProvider.java
@kvakvs
Copy link
Collaborator

kvakvs commented Dec 8, 2022

I am now running it with my work project for testing. Will report later.
Cannot build release due to 4 failed tests (i can skip tests too but they should be fixed for merge)
Screenshot 2022-12-08 at 07 29 27

@zyuyou
Copy link
Contributor Author

zyuyou commented Dec 9, 2022

I am now running it with my work project for testing. Will report later. Cannot build release due to 4 failed tests (i can skip tests too but they should be fixed for merge) Screenshot 2022-12-08 at 07 29 27

ok, I will check it out.

@zyuyou
Copy link
Contributor Author

zyuyou commented Dec 14, 2022

@kvakvs it's ok now

@kvakvs
Copy link
Collaborator

kvakvs commented Jan 2, 2023

Reviewed it twice and running it now, too. Great job, thank you. Will merge soon.

@kvakvs kvakvs merged commit b7bafd1 into ignatov:master Jan 3, 2023
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.

2 participants