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

Webstorm support #51

Merged
merged 3 commits into from
Oct 10, 2024
Merged

Conversation

nrayburn-tech
Copy link
Contributor

@nrayburn-tech nrayburn-tech commented Oct 9, 2024

  • Change the plugin dependencies to use the base platform (available in all IDEs) and the JavaScript module which is available for Webstorm or any IDE with the JavaScript plugin installed.
  • Update to run plugin verification for Webstorm and IntelliJ Ultimate

https://plugins.jetbrains.com/docs/intellij/plugin-compatibility.html#declaring-incompatibility-with-module

Depends on #50.

@@ -176,6 +176,9 @@ jobs:
name: Verify plugin
needs: [ build ]
runs-on: ubuntu-latest
strategy:
matrix:
platformType: ['IU', 'WS']
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This is so that we execute the verify step on both IntelliJ Ultimate and Webstorm. It ensures that the APIs we use in our plugin are compatible with both projects. I think we might also be able to only depend on Webstorm and assume that IntelliJ Ultimate contains everything from Webstorm and more.

I'm open to changing this PR around to function that way if that is the preferred approach.

@IWANABETHATGUY IWANABETHATGUY merged commit 492723c into oxc-project:main Oct 10, 2024
6 checks passed
@nrayburn-tech nrayburn-tech deleted the webstorm-support branch October 12, 2024 23:54
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