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: add dataflow to code issue data [IDE-134] #449

Merged
merged 1 commit into from
Feb 26, 2024

Conversation

bastiandoetsch
Copy link
Collaborator

@bastiandoetsch bastiandoetsch commented Feb 21, 2024

Description

In order to display the dataflow in the IDE custom UI (see screenshot below), we need additional data. This PR provides it via the $/snyk.scan notification. This is part of fixing issues for IDE-134.

Checklist

  • Tests added and all succeed
  • Linted
  • README.md updated, if user-facing
  • License file updated, if new 3rd-party dependency is introduced

🚨After having merged, please update the CLI go.mod to pull in latest language server.

Screenshots / GIFs

image

@bastiandoetsch bastiandoetsch requested a review from a team as a code owner February 21, 2024 08:25
@bastiandoetsch bastiandoetsch force-pushed the feat/IDE-134_add-dataflow-to-code-issue-data branch from fc3ee9f to 44b959f Compare February 21, 2024 11:21
@bastiandoetsch bastiandoetsch force-pushed the feat/IDE-134_add-dataflow-to-code-issue-data branch from 44b959f to f6c13b0 Compare February 21, 2024 11:22
@bastiandoetsch bastiandoetsch marked this pull request as draft February 21, 2024 11:26
@bastiandoetsch bastiandoetsch force-pushed the feat/IDE-134_add-dataflow-to-code-issue-data branch from f6c13b0 to b92568b Compare February 21, 2024 12:10
@bastiandoetsch bastiandoetsch force-pushed the feat/IDE-134_add-dataflow-to-code-issue-data branch from b92568b to 799a7c5 Compare February 21, 2024 12:20
@bastiandoetsch bastiandoetsch force-pushed the feat/IDE-134_add-dataflow-to-code-issue-data branch from 799a7c5 to 981de36 Compare February 21, 2024 12:21
@bastiandoetsch bastiandoetsch force-pushed the feat/IDE-134_add-dataflow-to-code-issue-data branch from 981de36 to 4e07811 Compare February 21, 2024 12:22
@bastiandoetsch bastiandoetsch force-pushed the feat/IDE-134_add-dataflow-to-code-issue-data branch from 4e07811 to b00b601 Compare February 21, 2024 12:41
This enables display of the dataflow in the custom UIs of the IDEs, as this object is transferred via the $/snyk.scan message.

The markers, used in VSCode, are not sufficient for this.
Copy link
Contributor

@acke acke left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me!

@bastiandoetsch bastiandoetsch merged commit f5ab230 into main Feb 26, 2024
15 checks passed
@bastiandoetsch bastiandoetsch deleted the feat/IDE-134_add-dataflow-to-code-issue-data branch February 26, 2024 10:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants