Skip to content

fix(mason-lspconfig): add missing field automatic_enable in setup #1527

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

Closed
wants to merge 1 commit into from

Conversation

SetsuikiHyoryu
Copy link

@SetsuikiHyoryu SetsuikiHyoryu commented May 12, 2025

I know this field is set by default, but if it's not explicitly written, Lua will throw an error:

Diagnostics:
1. Missing required fields in type `MasonLspconfigSettings`: `automatic_enable` [missing-fields]

@nickgnd
Copy link

nickgnd commented May 12, 2025

automatic_enable has been introduced in the latest version of Mason 2.0.

I think this change should be included in #1475 in case.

@SetsuikiHyoryu
Copy link
Author

automatic_enable has been introduced in the latest version of Mason 2.0.

I think this change should be included in #1475 in case.

@nickgnd I'm agree with you.

@carloscalla
Copy link
Contributor

This field automatic_enable was introduced in mason v2 but also automatic_installation and handlers were deprecated. As mentioned above, a more complete migration to mason v2 is being done in #1475

Please consider closing this PR to avoid confusion.

@SetsuikiHyoryu
Copy link
Author

Please consider closing this PR to avoid confusion.

Ok.

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.

3 participants