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

[pull] main from snyk:main #180

Merged
merged 3 commits into from
Nov 26, 2023
Merged

[pull] main from snyk:main #180

merged 3 commits into from
Nov 26, 2023

Conversation

pull[bot]
Copy link

@pull pull bot commented Nov 25, 2023

See Commits and Changes for more details.


Created by pull[bot]

Can you help keep this open source service alive? 💖 Please sponsor : )

cat2608 and others added 3 commits November 20, 2023 09:33
…EAD-975] (#392)

* fix: expand server settings returned by `LanguageClientMiddleware`

`LanguageClientMiddleware` includes new attributes such as `deviceId`, `integrationName`,
and `integrationVersion` to align with the initialization options.

BREAKING CHANGE: The middleware now requires a `User` object to be instantiated.

* refactor: merge server settings with initialization options

`ServerSettings` includes user-specific attributes such as `integrationName`,
`integrationVersion`, `automaticAuthentication`, and `deviceId`. This change
consolidates all configuration-related options into a single `ServerSettings` structure.

BREAKING CHANGE: The `fromConfiguration` method in `LanguageServerSettings` now requires `User` object.
All consumers of `ServerSettings` must now expect the additional user-specific properties.

* refactor: server settings and test assertions

Reworked ServerSettings to enhance clarity and logical grouping of configuration properties.
Added `defaultToTrue` utility function to handle undefined feature flags, defaulting them to 'true'
and ensuring consistent server settings initialization.

* chore: add CHANGELOG
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants