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

OTP27 support #1530

Merged
merged 5 commits into from
Sep 18, 2024
Merged

OTP27 support #1530

merged 5 commits into from
Sep 18, 2024

Conversation

MarkoMin
Copy link
Contributor

@MarkoMin MarkoMin commented Sep 13, 2024

Description

Things done:

  1. Completion for -doc and -moduledoc (+ tests)
  2. Bumped to new version of erlfmt which fully supports OTP27
  3. Removed some OTP23-compatibility code

Things to be done:

  1. Sigils completion
  2. Include OTP27 in CI

Things I need help with:

  1. Fixing els_diagnostics_SUITE - those even fail for me when I run them on current main branch with OTP25 (tescases fail due to "can't find include file" error, while test cases expect them to be found)
  2. Fixing els_completion_SUITE - some functions fail when comparing documentation (e.g. application_remote_otp- Seems like only signature is fetched as a markdown in OTP27, dunno exactly why. Any help here is very welcome

Fixes #1489.

@plux
Copy link
Contributor

plux commented Sep 16, 2024

Thanks for the contributions @MarkoMin . I see there seems. to be a problem with the build workflow, I will try to solve it.

@plux
Copy link
Contributor

plux commented Sep 16, 2024

@MarkoMin

Fix for the workflows is merged now.
Please re-base on main so we can re-run the tests.

@MarkoMin
Copy link
Contributor Author

seems like there is some issue with artifact caching in the workflow. Can you restart it?

@plux
Copy link
Contributor

plux commented Sep 18, 2024

Sorry, can you rebase again? It should be fixed now.

@plux plux merged commit eccc48d into erlang-ls:main Sep 18, 2024
2 of 5 checks passed
@plux
Copy link
Contributor

plux commented Sep 18, 2024

Thanks for you contribution @MarkoMin

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.

OTP 27 support
2 participants