-
Notifications
You must be signed in to change notification settings - Fork 7
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: create a new snyk view [IDE-711][IDE-712][IDE-713] #213
base: main
Are you sure you want to change the base?
Conversation
9083c23
to
e31bd11
Compare
e21aa41
to
07ecfa0
Compare
07ecfa0
to
5fc6f71
Compare
try { | ||
boolean cliFound = getCliFile().exists(); | ||
boolean tokenFound = !Preferences.getInstance().getAuthToken().isBlank() | ||
|| System.getenv().containsKey("SNYK_TOKEN"); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The SNYK_TOKEN check is old and a bug :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should we remove tokenFound, or only the System.getenv().containsKey check?
plugin/src/main/java/io/snyk/eclipse/plugin/views/snyktoolview/RootObject.java
Outdated
Show resolved
Hide resolved
plugin/src/main/java/io/snyk/eclipse/plugin/views/snyktoolview/RootObject.java
Outdated
Show resolved
Hide resolved
...c/main/java/io/snyk/eclipse/plugin/views/snyktoolview/handlers/EnableCodeProductHandler.java
Outdated
Show resolved
Hide resolved
27789d2
to
e027045
Compare
...c/main/java/io/snyk/eclipse/plugin/views/snyktoolview/handlers/EnableCodeProductHandler.java
Outdated
Show resolved
Hide resolved
def77c5
to
8d3a3c0
Compare
e17fecf
to
a460b03
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Leaving this in "Request Changes" as we'll use this as a base branch/PR for stacked PRs. Once we're ready, I'll move it to Approve :).
…m-IaC,-Open-Source-and-Code,-using-a-local-cache (#215) * chore: wip * fix: add info nodes for no issues found and no fixable issues found * fix: tests * feat: add initial tree filling impl * feat: display info nodes * feat: add generic filter method to cache * feat: show view when scanning
…s/expand treeviewer.
chore: UI cleanup [IDE-711 ]
Feat/fixable issues filter
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!