-
Notifications
You must be signed in to change notification settings - Fork 8
feat: fix lint #255
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: fix lint #255
Conversation
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.
Happy with this overall, though there are a few inconsistencies and further optimisations - see the comments.
The only thing that needs addressing before merge is your local path in the ruleset file (plugin/.pmd) - the rest are up to you.
plugin/src/main/java/io/snyk/eclipse/plugin/views/ScanWorkspaceFolderHandler.java
Show resolved
Hide resolved
plugin/src/main/java/io/snyk/eclipse/plugin/views/ScanWorkspaceMenuHandler.java
Show resolved
Hide resolved
plugin/src/main/java/io/snyk/eclipse/plugin/views/snyktoolview/filters/BaseFilter.java
Outdated
Show resolved
Hide resolved
public void selectionChanged(IAction action, ISelection selection) { | ||
this.selection = selection; | ||
} | ||
IWorkbenchPart part; |
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.
Inconsistent indentation compared to the file above. Which is correct?
plugin/src/main/java/io/snyk/languageserver/LsConfigurationUpdater.java
Outdated
Show resolved
Hide resolved
plugin/.ruleset
Outdated
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.
We should be able to automatically generate it from our "master" XML file in the root. But that's something for a different PR.
Description
This PR solves a number of lint errors.
Checklist
Screenshots / GIFs
Visuals that may help the reviewer. Please add screenshots for any UI change. GIFs are most welcome!