Conversation
There was a problem hiding this comment.
Pull request overview
This PR bumps the language server dependencies from version 0.3.30 to 0.3.46 and enables the case function feature by default through the experimentalFeatures.allowCaseFunction configuration option. The update brings 16 minor versions of improvements to the language server and workflow parser libraries.
Changes:
- Upgraded
@actions/languageserverand@actions/workflow-parserfrom ^0.3.30 to ^0.3.46 - Enabled case function support via experimentalFeatures configuration in language server initialization
Reviewed changes
Copilot reviewed 2 out of 3 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| src/workflow/languageServer.ts | Added experimentalFeatures configuration with allowCaseFunction set to true |
| package.json | Bumped @actions/languageserver and @actions/workflow-parser to ^0.3.46 |
| package-lock.json | Updated dependency tree with new versions and transitive dependencies |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| "@actions/languageserver": "^0.3.46", | ||
| "@actions/workflow-parser": "^0.3.46", |
There was a problem hiding this comment.
The updated language server dependencies (@actions/languageserver and @actions/workflow-parser version 0.3.46) require Node >= 20 according to the package-lock.json, but the package.json still specifies Node >= 16 in the engines field (line 14). This creates a compatibility mismatch. Either the engines field should be updated to require Node >= 20, or a version of the dependencies that supports Node >= 16 should be used instead.
| "@actions/languageserver": "^0.3.46", | |
| "@actions/workflow-parser": "^0.3.46", | |
| "@actions/languageserver": "^0.3.44", | |
| "@actions/workflow-parser": "^0.3.44", |
Enable case function by default with the latest version of language server.