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

Diagnostics: Change nwnsc for nwn_script_comp #55

Open
WilliamDraco opened this issue Jul 19, 2023 · 1 comment · May be fixed by #62
Open

Diagnostics: Change nwnsc for nwn_script_comp #55

WilliamDraco opened this issue Jul 19, 2023 · 1 comment · May be fixed by #62
Assignees
Projects
Milestone

Comments

@WilliamDraco
Copy link

Recently, the official compiler has been open-sourced as part of the neverwinter.nim project. This compiler matches the toolset capabilities, and updates to the project will be reflected back into the toolset compiler over time where relevant.

Importantly, some new language features are not supported by nwnsc and report errors which are no longer true: For example, Switch-Case statements can now take strings, but nwnsc cannot handle that and reports errors. Swapping to nwn_script_comp would ensure it maintains correct diagnostics/compatibility with any additional features added.

@PhilippeChab PhilippeChab self-assigned this Jul 19, 2023
@PhilippeChab PhilippeChab added this to To do in LSP via automation Jul 19, 2023
@PhilippeChab PhilippeChab added this to the v2.2.0 milestone Jul 19, 2023
@PhilippeChab PhilippeChab linked a pull request Oct 22, 2023 that will close this issue
2 tasks
@PhilippeChab
Copy link
Owner

I've opened a PR to make the change, unfortunately there are two points that need to be addressed before we can go ahead with it. See the PR for the details.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
LSP
To do
Development

Successfully merging a pull request may close this issue.

2 participants