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

refactor: rename common classes from SnykCode prefix [IDE-283] #527

Merged
merged 2 commits into from
May 10, 2024

Conversation

teodora-sandu
Copy link
Contributor

Description

Renames and moves common "LS in IntelliJ" files that were used when integrating Snyk Code via LS, so that we can use them when we integrated Snyk OSS via LS.

There is not actual change but I have tested it in IntelliJ by running the IDE, just to sanity check that Snyk Code scanning still works.

Checklist

  • Tests added and all succeed
  • Linted
  • CHANGELOG.md updated
  • README.md updated, if user-facing

Screenshots

Screenshot 2024-05-09 at 14 09 49

@teodora-sandu teodora-sandu requested a review from a team as a code owner May 9, 2024 13:10
@teodora-sandu teodora-sandu changed the title refactor: rename common classes from SnykCode prefix refactor: rename common classes from SnykCode prefix [IDE-283] May 9, 2024
Copy link
Contributor

@cat2608 cat2608 left a comment

Choose a reason for hiding this comment

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

Tested it locally and it works fine, thank you 🙌
image

@teodora-sandu teodora-sandu merged commit 662e041 into master May 10, 2024
9 checks passed
@teodora-sandu teodora-sandu deleted the refactor/snyk-code-ls-common-files branch May 10, 2024 13:02
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.

2 participants