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

feat(language-server): integrate LS #5672

Merged
merged 1 commit into from
Jan 20, 2025

Conversation

Team-Hammerhead
Copy link
Collaborator

@Team-Hammerhead Team-Hammerhead commented Jan 20, 2025

Changes since last integration of Language Server

commit b692957e61a8f3db863429e34f8198ecb6697776
Author: Knut Funkel <knut.funkel@snyk.io>
Date:   Mon Jan 20 13:29:33 2025 +0100

    feat: add a new $/snyk.scanSummary notificiation [IDE-868] (#758)

:100644 100644 fcebb3ed 1fdf21f6 M	README.md
:100644 100644 eff30dbc 434a812f M	application/server/notification.go
:100644 100644 f7190346 c9c70a01 M	internal/types/lsp.go

commit 5facb0f22c1109f867d692db2a8930dbb6d94951
Author: Zdroba <72391375+DariusZdroba@users.noreply.github.com>
Date:   Mon Jan 20 12:38:54 2025 +0200

    fix: fix issue metadata used for hashing [IDE-816] (#757)

:100644 100644 b5d603c9 d3e8aee2 M	infrastructure/utils/fingerprint.go

@Team-Hammerhead Team-Hammerhead requested a review from a team as a code owner January 20, 2025 11:54
Copy link
Contributor

github-actions bot commented Jan 20, 2025

Warnings
⚠️

"feat: automatic integration of language server b692957e61a8f3db863429e34f8198ecb6697776" is too long. Keep the first line of your commit message under 72 characters.

Generated by 🚫 dangerJS against fc80c9c

@team-ide-user team-ide-user force-pushed the feat/automatic-upgrade-of-ls branch from 6ef0930 to fc80c9c Compare January 20, 2025 12:38
@Team-Hammerhead Team-Hammerhead merged commit 7ce5630 into main Jan 20, 2025
9 checks passed
@Team-Hammerhead Team-Hammerhead deleted the feat/automatic-upgrade-of-ls branch January 20, 2025 13:17
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