Skip to content

Add JS/TS configuration #32

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

Merged
merged 1 commit into from
May 4, 2023

Conversation

lucas-paulger-sonarsource
Copy link
Contributor

Ismail and I ran into some configuration needs when setting up IntelliJ to run the web project.

Copy link

@ismailcherri ismailcherri left a comment

Choose a reason for hiding this comment

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

LGTM ✨

@antonio-garcia-sonarsource

There's actually a solution for Prettier that doesn't require a plugin and it is what I am using. Did you consider it? I am trying the plugin now to see how it compares

image

@lucas-paulger-sonarsource
Copy link
Contributor Author

There's actually a solution for Prettier that doesn't require a plugin and it is what I am using. Did you consider it? I am trying the plugin now to see how it compares

Let me know what you think, I did not try the non-plugin route.

@ismailcherri
Copy link

There's actually a solution for Prettier that doesn't require a plugin and it is what I am using. Did you consider it? I am trying the plugin now to see how it compares

I prefer the plugin approach because it is easier to setup

@gregaubert
Copy link
Member

Looks good from my point of view! It might be worth pinging SQ, SonarLint or JS/TS bubble to know if any of them are also using IntelliJ and get their opinion on the config

@lucas-paulger-sonarsource lucas-paulger-sonarsource merged commit 05a361e into master May 4, 2023
@lucas-paulger-sonarsource lucas-paulger-sonarsource deleted the feature/lp/add-js-ts-config branch May 4, 2023 07:19
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