Skip to content

Conversation

@nick-y-snyk
Copy link
Contributor

@nick-y-snyk nick-y-snyk commented Nov 14, 2025

…cation when Snyk Code isn't enabled for folder
image

Description

Provide description of this PR and changes, if linked Jira ticket doesn't cover it in full.

Checklist

Screenshots / GIFs

Visuals that may help the reviewer. Please add screenshots for any UI change. GIFs are most welcome!

…cation when Snyk Code isn't enabled for folder
@nick-y-snyk nick-y-snyk requested review from a team as code owners November 14, 2025 12:54
@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@snyk-io
Copy link

snyk-io bot commented Nov 14, 2025

Snyk checks have passed. No issues have been found so far.

Status Scanner Critical High Medium Low Total (0)
Code Security 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

getSnykCachedResults(project)?.currentSnykCodeError != null -> {
val errorMessage = getSnykCachedResults(project)?.currentSnykCodeError?.message
val errorSuffix = when {
errorMessage?.contains("not enabled", ignoreCase = true) == true -> "(disabled at Snyk)"
Copy link
Contributor Author

Choose a reason for hiding this comment

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

put the whole string or send another param from ls

this.snykToolWindowPanel.displayScanningMessage()
}

this.snykToolWindowPanel.cleanUiAndCaches()
Copy link
Contributor Author

Choose a reason for hiding this comment

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

add TODO to clear cache per product

Copy link
Contributor

@rrama rrama left a comment

Choose a reason for hiding this comment

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

Please do the changes in the comment, then it's good.
Oh, and update your PR title please.

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.

4 participants